Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

adds unknown config and abbrev handlers #180

Merged
merged 2 commits into from
Jan 21, 2025
Merged

Conversation

wraithgar
Copy link
Member

@wraithgar wraithgar commented Jan 8, 2025

This is part of npm/statusboard#491
It also reverts the changes in #179 which were never published.

  • deps: remove proc-log
  • feat: add unknownHandler and abbrevHandler

Closes: #41

Example npm output using these new handlers:

output of npm consuming this change logging warnings from all of these new handlers

We are going a different way here
@wraithgar wraithgar requested a review from a team as a code owner January 8, 2025 18:00
@wraithgar wraithgar marked this pull request as draft January 9, 2025 18:23
@wraithgar wraithgar force-pushed the gar/unknown-handler branch from 94b5188 to e2c556d Compare January 10, 2025 16:16
@wraithgar wraithgar changed the title gar/unknown handler adds unknown config and abbrev handlers Jan 10, 2025
@wraithgar wraithgar marked this pull request as ready for review January 10, 2025 16:19
@wraithgar wraithgar marked this pull request as draft January 10, 2025 18:47
@wraithgar wraithgar force-pushed the gar/unknown-handler branch from e2c556d to 5af2372 Compare January 13, 2025 17:23
@wraithgar wraithgar marked this pull request as ready for review January 13, 2025 17:30
@wraithgar wraithgar force-pushed the gar/unknown-handler branch from 5af2372 to 297581a Compare January 13, 2025 17:34
This is in favor of proc-log
@wraithgar wraithgar force-pushed the gar/unknown-handler branch from 297581a to 84ff74c Compare January 14, 2025 16:34
lib/nopt-lib.js Show resolved Hide resolved
@wraithgar wraithgar merged commit 3319782 into main Jan 21, 2025
15 checks passed
@wraithgar wraithgar deleted the gar/unknown-handler branch January 21, 2025 17:06
@github-actions github-actions bot mentioned this pull request Jan 10, 2025
wraithgar pushed a commit that referenced this pull request Jan 21, 2025
🤖 I have created a release *beep* *boop*
---


## [8.1.0](v8.0.0...v8.1.0)
(2025-01-21)
### Features
*
[`3319782`](3319782)
[#180](#180) add unknownHandler and
abbrevHandler (@wraithgar)
### Bug Fixes
*
[`304b903`](304b903)
[#179](#179) log warnings about unknown
config situations (@wraithgar)
### Dependencies
*
[`a7156b2`](a7156b2)
[#180](#180) remove proc-log
*
[`8dd86dd`](8dd86dd)
[#179](#179) add `proc-log@5.0.0`
*
[`9e64314`](9e64314)
[#176](#176) bump abbrev from 2.0.0 to
3.0.0 (#176)
### Chores
*
[`d7916cd`](d7916cd)
[#181](#181) bump @npmcli/template-oss
from 4.23.4 to 4.23.6 (#181) (@dependabot[bot], @npm-cli-bot)

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] Add a handler for *unknown* options, analogous to invalidHandler
2 participants