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.

Sign up for GitHub

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

Jump to bottom

In the test suite, run arbitrary "lint" checks on code #200

Closed
julianhyde opened this issue Aug 20, 2023 · 1 comment
Closed

In the test suite, run arbitrary "lint" checks on code #200

julianhyde opened this issue Aug 20, 2023 · 1 comment

Comments

@julianhyde
Copy link
Collaborator

Add a test class LintTest that runs arbitrary checks on the source code.

The test is similar to the LintTest added in CALCITE-5765, and also uses the Puffin script engine, added in CALCITE-5764.

This change will require upgrading to Calcite 1.35, because of the dependency on Puffin.
julianhyde added a commit that referenced this issue Aug 20, 2023
@julianhyde
Code style: Lint
b36504b 
Fix lint violations detected by #200
julianhyde added a commit that referenced this issue Aug 20, 2023
@julianhyde
Code style: Lint
c775461 
Fix lint violations detected by #200
@julianhyde
Copy link
Collaborator Author

julianhyde commented Aug 20, 2023 •
edited
Loading

Work in progress: https://github.com/julianhyde/morel/tree/200-lint

julianhyde added a commit to julianhyde/morel that referenced this issue Nov 11, 2023
@julianhyde
Code style: Lint
f61a46e 
Fix lint violations detected by hydromatic#200
julianhyde added a commit to julianhyde/morel that referenced this issue Nov 13, 2023
@julianhyde
Code style: Lint
50bd631 
Fix lint violations detected by hydromatic#200
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@julianhyde