Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Refactor help command implementation to hold actual Command #2087

Merged
merged 19 commits into from
Jan 11, 2024
Merged

Refactor help command implementation to hold actual Command #2087

merged 19 commits into from
Jan 11, 2024

Conversation

shadowspawn
Copy link
Collaborator

@shadowspawn shadowspawn commented Nov 26, 2023 •
edited
Loading

Pull Request

Problem

In order of importance:

  1. the other addFoo routines like addCommand() take objects
  2. we have to do custom work and add more properties for new Command features, like say helpGroup (Add helpGroup for options and commands #1910)
  3. bit messy storing multiple properties to construct help command when needed (minor)

Solution

  • make .addHelpCommand() accept a Command
    • so similar to .addCommand()
  • add .helpCommand() which behaves similarly to the old API
    • so similar to .command()
  • .addHelpCommand() still accepts string/boolean/undefined for backwards compatibility, but that usage is deprecated in favour of .helpCommand()

ChangeLog

  • added: .helpCommand() for configuring built-in help command
  • changed: .addHelpCommand() now takes a Command for configuring built-in help command
  • deprecated: .addHelpCommand() being passed string or boolean for configuring built-in help command

dependabot bot and others added 11 commits November 11, 2023 22:18
@dependabot
Bump @types/jest from 29.5.6 to 29.5.7 (tj#2064)
48f9fd9 
Bumps [@types/jest](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/jest) from 29.5.6 to 29.5.7.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/jest)

---
updated-dependencies:
- dependency-name: "@types/jest"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump eslint from 8.52.0 to 8.53.0 (tj#2061)
902124d 
Bumps [eslint](https://github.com/eslint/eslint) from 8.52.0 to 8.53.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/main/CHANGELOG.md)
- [Commits](eslint/eslint@v8.52.0...v8.53.0)

---
updated-dependencies:
- dependency-name: eslint
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump @types/node from 20.8.9 to 20.8.10 (tj#2063)
c20a71d 
Bumps [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node) from 20.8.9 to 20.8.10.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node)

---
updated-dependencies:
- dependency-name: "@types/node"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump @typescript-eslint/eslint-plugin from 6.9.0 to 6.9.1 (tj#2060)
6b74d26 
Bumps [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) from 6.9.0 to 6.9.1.
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v6.9.1/packages/eslint-plugin)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/eslint-plugin"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump @typescript-eslint/parser from 6.9.0 to 6.9.1 (tj#2062)
2978456 
Bumps [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser) from 6.9.0 to 6.9.1.
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/parser/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v6.9.1/packages/parser)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/parser"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump @types/jest from 29.5.7 to 29.5.8 (tj#2076)
883d326 
Bumps [@types/jest](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/jest) from 29.5.7 to 29.5.8.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/jest)

---
updated-dependencies:
- dependency-name: "@types/jest"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump @types/node from 20.8.10 to 20.9.0 (tj#2074)
f234c25 
Bumps [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node) from 20.8.10 to 20.9.0.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node)

---
updated-dependencies:
- dependency-name: "@types/node"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump @typescript-eslint/eslint-plugin from 6.9.1 to 6.10.0 (tj#2072)
c1b2526 
Bumps [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) from 6.9.1 to 6.10.0.
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v6.10.0/packages/eslint-plugin)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/eslint-plugin"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump @typescript-eslint/parser from 6.9.1 to 6.10.0 (tj#2073)
47a48c9 
Bumps [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser) from 6.9.1 to 6.10.0.
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/parser/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v6.10.0/packages/parser)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/parser"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump eslint-plugin-n from 16.2.0 to 16.3.1 (tj#2075)
8eed8c2 
Bumps [eslint-plugin-n](https://github.com/eslint-community/eslint-plugin-n) from 16.2.0 to 16.3.1.
- [Release notes](https://github.com/eslint-community/eslint-plugin-n/releases)
- [Commits](eslint-community/eslint-plugin-n@16.2.0...16.3.1)

---
updated-dependencies:
- dependency-name: eslint-plugin-n
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@shadowspawn
Add .helpCommand and store custom command
7685a2b
@shadowspawn shadowspawn changed the base branch from master to develop November 26, 2023 05:47
@shadowspawn shadowspawn changed the title Experiment: refactor help command implementation to hold actual Command Refactor help command implementation to hold actual Command Jan 3, 2024
@shadowspawn shadowspawn marked this pull request as ready for review January 6, 2024 01:03
@shadowspawn shadowspawn added the semver: major Releasing requires a major version bump, not backwards compatible label Jan 6, 2024
@shadowspawn shadowspawn changed the base branch from develop to release/12.x January 6, 2024 01:33
abetomo
abetomo reviewed Jan 10, 2024
View reviewed changes
lib/command.js Outdated Show resolved Hide resolved
abetomo
abetomo approved these changes Jan 10, 2024
View reviewed changes
Copy link
Collaborator

@abetomo abetomo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@shadowspawn shadowspawn merged commit ff08a02 into tj:release/12.x Jan 11, 2024
6 checks passed
@shadowspawn shadowspawn deleted the feature/help-command-as-property branch January 11, 2024 04:34
@shadowspawn shadowspawn mentioned this pull request Jan 14, 2024
Closed
@shadowspawn shadowspawn added the pending release Merged into a branch for a future release, but not released yet label Jan 19, 2024
@schwer schwer mentioned this pull request Jul 6, 2024
Closed
@JhayceFrancis JhayceFrancis mentioned this pull request Jul 16, 2024
Open
This was referenced Jul 16, 2024
Open
Open
Open
Open
Open
Open
@Ramyromel Ramyromel mentioned this pull request Aug 10, 2024
Merged
@Exkaleburx Exkaleburx mentioned this pull request Sep 6, 2024
Open
@q1blue q1blue mentioned this pull request Sep 8, 2024
Open
This was referenced Sep 11, 2024
Open
Open
Open
Open
@AKJUS AKJUS mentioned this pull request Sep 12, 2024
Open
@q1blue q1blue mentioned this pull request Sep 13, 2024
Open
This was referenced Sep 14, 2024
Open
Open
Open
Open
Open
@q1blue q1blue mentioned this pull request Sep 20, 2024
Open
@WontonSam WontonSam mentioned this pull request Sep 22, 2024
Open
@q1blue q1blue mentioned this pull request Sep 22, 2024
Open
@snyk-io snyk-io bot mentioned this pull request Sep 23, 2024
Open
@jihaui jihaui mentioned this pull request Sep 25, 2024
Open
@WontonSam WontonSam mentioned this pull request Oct 15, 2024
Open
@lholmquist lholmquist mentioned this pull request Oct 31, 2024
Open
@snyk-io snyk-io bot mentioned this pull request Nov 9, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@abetomo abetomo abetomo approved these changes

Assignees
No one assigned
Labels
semver: major Releasing requires a major version bump, not backwards compatible
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@shadowspawn @abetomo