Skip to content

chore: improve argument parsing#213

Merged
aladdin-add merged 6 commits intoeslint:mainfrom
hyperz111:parse-args
Feb 1, 2026
Merged

chore: improve argument parsing#213
aladdin-add merged 6 commits intoeslint:mainfrom
hyperz111:parse-args

Conversation

@hyperz111
Copy link
Copy Markdown
Contributor

@hyperz111 hyperz111 commented Oct 22, 2025

Prerequisites checklist

What is the purpose of this pull request?

To improve argument parsing.

What changes did you make? (Give an overview)

Using parseArgs method from node:util module.

Now we can do this (using equal mark):

npm create @eslint/config --config=eslint-config-airbnb

Related Issues

N/A

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

Maybe we can remove n/unsupported-features/node-builtins when #206 is merged.

@eslintbot eslintbot added this to Triage Oct 22, 2025
@github-project-automation github-project-automation Bot moved this to Needs Triage in Triage Oct 22, 2025
@aladdin-add aladdin-add moved this from Needs Triage to Blocked in Triage Oct 23, 2025
@aladdin-add aladdin-add self-assigned this Oct 23, 2025
@aladdin-add aladdin-add marked this pull request as draft October 23, 2025 08:53
@hyperz111
Copy link
Copy Markdown
Contributor Author

@aladdin-add, why you draft it?

@aladdin-add
Copy link
Copy Markdown
Member

Not sure if utils.parseArgs is supported in node v18, I would merge it after #206 merged (for safety).

@aladdin-add aladdin-add marked this pull request as ready for review January 31, 2026 15:03
Copilot AI review requested due to automatic review settings January 31, 2026 15:03
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread bin/create-config.js Outdated
@hyperz111
Copy link
Copy Markdown
Contributor Author

@aladdin-add, this is ready to merged.

Copy link
Copy Markdown
Member

@aladdin-add aladdin-add 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!

@aladdin-add aladdin-add merged commit a10a765 into eslint:main Feb 1, 2026
17 checks passed
@github-project-automation github-project-automation Bot moved this from Blocked to Complete in Triage Feb 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

4 participants