[ Web Proxy ]
URL:
Viewing: https://github-redirect.dependabot.com/log4js-node/log4js-node/pull/1082 [Back]  [Original]

Shutdown cb arg can be undefined + always return null by richardhinkamp Pull Request #1082 log4js-node/log4js-node GitHub
Skip to content

Navigation Menu

Repository navigation

More items

Shutdown cb arg can be undefined + always return null - #1082

Merged
lamweili merged 2 commits into
log4js-node:masterfrom
richardhinkamp:shutdown-type-consistent
Sep 2, 2022
Merged

lamweili merged 2 commits into
log4js-node:masterfrom
richardhinkamp:shutdown-type-consistent

Conversation

@richardhinkamp [@richardhinkamp]

Copy link
Copy Markdown
Contributor

Make shutdown consistent:

  • cb is optional in both function definitions
  • cb can get undefined as arg when no error occured
  • with no shutdown functions, return null instead of the return value of cb(), like the return value with shutdown functions.
  • with no shutdown functions, explicit pass of undefined to cb() for simpler typings of cb() to always have an argument.
[Go]
@lamweili [@lamweili] lamweili added this to the 6.7.0 milestone Sep 2, 2022
lamweili merged commit 6a60294 into log4js-node:master Sep 2, 2022
lamweili added a commit that referenced this pull request Sep 3, 2022
refactor(#1082): removed return value for `log4js.shutdown()`
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Footer

© 2026 GitHub, Inc.

Web Proxy Viewer  |  New URL  |  Original Page