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

feat: support Bazel buildifier linter #4529

Merged
merged 3 commits into from
Jun 27, 2023

Conversation

cgrindel
Copy link
Contributor

@cgrindel cgrindel commented May 30, 2023

Support linter messages using Bazel buildifier. The linter support leverages existing capabilities introduced with the fixer implementation.

@cgrindel
Copy link
Contributor Author

@w0rp @hsanson Gentle ping for a review.

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.

@hsanson hsanson merged commit 21f1ab6 into dense-analysis:master Jun 27, 2023
6 checks passed
@cgrindel cgrindel deleted the support_buildifier_lint_msgs branch June 27, 2023 19:44
mnikulin pushed a commit to mnikulin/ale that referenced this pull request Nov 12, 2023
* Initial buildifier linter files

* Add handler test

* Fix test when options are not set
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