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.

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

Expose minimist #661

Merged
merged 5 commits into from Aug 15, 2023
Merged

Expose minimist #661

merged 5 commits into from Aug 15, 2023

Conversation

LionC
Copy link
Contributor

@LionC LionC commented Aug 14, 2023

Fixes #553

  • Tests pass
  • Appropriate changes to README are included in PR
  • Types updated

src/globals.ts Outdated Show resolved Hide resolved
@LionC LionC changed the title Expose minimist/553 Expose minimist / solves #553 Aug 14, 2023
@LionC LionC changed the title Expose minimist / solves #553 Expose minimist Aug 14, 2023
@antongolub
Copy link
Contributor

@LionC,

Could you rebase your branch?

@LionC
Copy link
Contributor Author

LionC commented Aug 15, 2023

@LionC,

Could you rebase your branch?

Done :-)

Copy link
Contributor

@antongolub antongolub left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for the improvement.

@antonmedv antonmedv merged commit b1ca232 into google:main Aug 15, 2023
9 checks passed
@LionC
Copy link
Contributor Author

LionC commented Aug 16, 2023

No problem - looking forward to use it in out scripts. Do you already know when it will be released?

@antonmedv
Copy link
Collaborator

Soon)

@LionC LionC deleted the expose-minimist/553 branch August 16, 2023 20:14
@nomyfan
Copy link

nomyfan commented Nov 1, 2023

It has been months, any release plan?

@aarondill
Copy link

aarondill commented Nov 16, 2023

It has been months, any release plan?

Wondering this same thing?

@milesrichardson
Copy link

Oh, this isn't released? That explains why this wasn't working for me (despite having the apparently latest version 7.2.3).

The frustrating part is that the docs suggest that it has been released, as they show an example using minimist.

milesrichardson added a commit to milesrichardson/crxmon that referenced this pull request Nov 20, 2023
…t exported

Despite the `zx` docs suggesting that `minimist` is exported from `zx`,
that's not actually true. The code has been pushed for this feature, but
it's not released to `npm` yet.

Accordingly, manually install `minimist` so that we can import from it.
Be careful to match the exact version (`1.2.8`) which was resolved when
installing it as a dependency of `zx` (discovered through `yarn info -R minimist`
and then added with `yarn add -E minimist@1.2.8`).

See: google/zx#661
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature request: Configuration of minimist
6 participants