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

ci: add linting and formatting automation to tree-sitter grammar #68

Merged
merged 5 commits into from
Mar 25, 2024

Conversation

ttytm
Copy link
Member

@ttytm ttytm commented Mar 25, 2024

The PR ensures better code consistency and standards.

  • For linting it adds eslint (commonly used for linting in js projects and by official tree-sitter grammars, e.g., rust, go, bash).
  • It adds a prettier config file.
  • Adds steps to run linting and verify formatting to the tree-sitter workflow.

The PR makes uses of the above to fix lint warnings and to format the code.
To verify those fixes and the formatting didn't change the grammar, the current parser.c can be re-generated and confirmed that it didn't change.

remove obsolet link filelinks, preserve tree-sitter convetion, commonjs is specified in eslint

s
@ttytm ttytm merged commit da0005b into vlang:main Mar 25, 2024
13 checks passed
@ttytm ttytm deleted the ci/eslint branch March 25, 2024 21:41
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.

1 participant