Skip to content

chore(style,grammar) add eslint configuration file#238

Merged
fdncred merged 8 commits into
nushell:mainfrom
mkatychev:chore/eslint-formatter
Dec 13, 2025
Merged

chore(style,grammar) add eslint configuration file#238
fdncred merged 8 commits into
nushell:mainfrom
mkatychev:chore/eslint-formatter

Conversation

@mkatychev
Copy link
Copy Markdown
Contributor

@mkatychev mkatychev commented Dec 10, 2025

Most tree-sitter eslint configurations do not include eslint-prettier which I suspect is why some tree-sitter-eslint style guides were being overridden.

This PR removes the prettier config and file and applies formatting to resembled more popular tree-sitter grammars.

https://github.com/tree-sitter/tree-sitter-rust/blob/master/eslint.config.mjs

https://github.com/tree-sitter/tree-sitter-rust/blob/261b20226c04ef601adbdf185a800512a5f66291/grammar.js#L129
https://github.com/tree-sitter/tree-sitter-rust/blob/master/grammar.js#L136

@fdncred
Copy link
Copy Markdown
Contributor

fdncred commented Dec 10, 2025

kind of a weird ci error

@mkatychev
Copy link
Copy Markdown
Contributor Author

I'm not able to rerun the workflow, I'll push a commit once back at my desk

@mkatychev mkatychev force-pushed the chore/eslint-formatter branch from fc615e5 to 3dac2bf Compare December 12, 2025 06:47
@mkatychev mkatychev force-pushed the chore/eslint-formatter branch from 3dac2bf to 59e3ba2 Compare December 12, 2025 06:49
@fdncred fdncred merged commit 4c14962 into nushell:main Dec 13, 2025
4 checks passed
@fdncred
Copy link
Copy Markdown
Contributor

fdncred commented Dec 13, 2025

Thanks

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.

2 participants