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

Process --version Flag Directly #264

Merged
merged 3 commits into from Oct 4, 2022
Merged

Conversation

lindluni
Copy link
Contributor

@lindluni lindluni commented Oct 4, 2022

What is the purpose of this pull request? (put an "X" next to item)

[ ] Documentation update
[x] Bug fix
[ ] New feature
[ ] Other, please explain:

What changes did you make? (Give an overview)

Directly process the --version flag as per: #263

Which issue (if any) does this pull request address?

#263

Is there anything you'd like reviewers to focus on?

Processing the version flag directly in the cli.js module much like the eslint project does: https://github.com/eslint/eslint/blob/main/lib/cli.js#L333

@welcome
Copy link

welcome bot commented Oct 4, 2022

🙌 Thanks for opening this pull request! You're awesome.

@lindluni lindluni changed the title Process version flag directly Process --version Flag Directly Oct 4, 2022
Copy link
Member

@theoludwig theoludwig 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! 👍

Could you add a test in test/cli.js to avoid regressions in the future?

@lindluni
Copy link
Contributor Author

lindluni commented Oct 4, 2022

Hi @divlo

I've added the test, thanks for pointing this out! Appreciate the help!

@theoludwig theoludwig merged commit eee0fb5 into standard:master Oct 4, 2022
@welcome
Copy link

welcome bot commented Oct 4, 2022

🎉 Congrats on getting your first pull request landed!

@lindluni
Copy link
Contributor Author

lindluni commented Oct 4, 2022

@divlo

Any chance we can get a 12.0.1 tag and publish it to npm to pick up this fix?

Thank you for your help with this!

@theoludwig
Copy link
Member

@lindluni

PR for the release available there: #265.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

2 participants