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

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] <support@github.com>
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] <support@github.com>
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] <support@github.com>
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] <support@github.com>
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] <support@github.com>
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] <support@github.com>
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] <support@github.com>
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] <support@github.com>
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] <support@github.com>
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] <support@github.com>
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
@OKEAMAH OKEAMAH mentioned this pull request May 27, 2024
Open
This was referenced Jun 6, 2024
Open
Merged
@Exkaleburx Exkaleburx mentioned this pull request Jun 8, 2024
Open
@viqueen viqueen mentioned this pull request Jun 8, 2024
Closed
@OKEAMAH OKEAMAH mentioned this pull request Jun 9, 2024
Open
@nicomt nicomt mentioned this pull request Jun 9, 2024
Open
@calvin-kimani calvin-kimani mentioned this pull request Jun 9, 2024
Open
@StemmlerSisters StemmlerSisters mentioned this pull request Jun 9, 2024
Open
@OKEAMAH OKEAMAH mentioned this pull request Jun 9, 2024
Open
@Noggling Noggling mentioned this pull request Jun 10, 2024
Closed
This was referenced Jun 19, 2024
Open
Open
@JoKacar JoKacar mentioned this pull request Jun 23, 2024
Merged
@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
@OKEAMAH OKEAMAH mentioned this pull request Aug 2, 2024
Open
@Ramyromel Ramyromel mentioned this pull request Aug 10, 2024
Merged
This was referenced Aug 16, 2024
Open
Open
Open
Open
Open
Open
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.

None yet
2 participants
@shadowspawn @abetomo