Skip to content

tools: consolidate and simplify .editorconfig deps section#62887

Closed
watilde wants to merge 1 commit intonodejs:mainfrom
watilde:tools-editorconfig
Closed

tools: consolidate and simplify .editorconfig deps section#62887
watilde wants to merge 1 commit intonodejs:mainfrom
watilde:tools-editorconfig

Conversation

@watilde
Copy link
Copy Markdown
Member

@watilde watilde commented Apr 22, 2026

Separate deps/** into its own .editorconfig section and add insert_final_newline = unset to avoid applying project formatting rules to vendored dependencies.

@nodejs-github-bot nodejs-github-bot added the tools Issues and PRs related to the tools directory. label Apr 22, 2026
@watilde watilde added the request-ci Add this label to start a Jenkins CI on a PR. label Apr 23, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Apr 23, 2026
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@watilde watilde force-pushed the tools-editorconfig branch 2 times, most recently from 1df4e04 to fe8397e Compare April 23, 2026 11:07
@watilde
Copy link
Copy Markdown
Member Author

watilde commented Apr 23, 2026

rebase and make the commit Verified.

@watilde watilde added the request-ci Add this label to start a Jenkins CI on a PR. label Apr 23, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Apr 23, 2026
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@watilde watilde added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Apr 25, 2026
watilde added a commit that referenced this pull request Apr 26, 2026
PR-URL: #62887
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@watilde
Copy link
Copy Markdown
Member Author

watilde commented Apr 26, 2026

landed in 11f76b4

@watilde watilde closed this Apr 26, 2026
@watilde watilde deleted the tools-editorconfig branch April 26, 2026 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. tools Issues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants