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

add rego support #4199

Merged
merged 4 commits into from
May 16, 2022
Merged

add rego support #4199

merged 4 commits into from
May 16, 2022

Conversation

rzjfr
Copy link
Contributor

@rzjfr rzjfr commented May 12, 2022

Where are the tests? Have you added tests? Have you updated the tests?

test/fixers/test_opa_fmt_fixer_callback.vader
test/linter/test_rego_opacheck.vader

Related issue:

#2531

@rzjfr rzjfr changed the title add rego add rego support May 12, 2022
@rzjfr rzjfr marked this pull request as ready for review May 13, 2022 10:21
doc/ale.txt Show resolved Hide resolved
Copy link
Contributor

@hsanson hsanson left a comment

Choose a reason for hiding this comment

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

Looks good and complete. Just missing cspell support in the list of supported tools for rego file.

Copy link
Contributor

@hsanson hsanson left a comment

Choose a reason for hiding this comment

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

Thanks for the prompt fixes.

@hsanson hsanson merged commit 75d2413 into dense-analysis:master May 16, 2022
cyyever pushed a commit to cyyever/ale that referenced this pull request Jul 11, 2022
* add opa fmt fixer for rego files

* add opa linter

* add basic tests for linter and fixer

* add cspell to the docs
cyyever pushed a commit to cyyever/ale that referenced this pull request Jul 11, 2022
* add opa fmt fixer for rego files

* add opa linter

* add basic tests for linter and fixer

* add cspell to the docs
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.

None yet

2 participants