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

Remove ktlint IJ plugin from required plugins #451

Merged
merged 1 commit into from
Jul 17, 2024
Merged

Conversation

rock3r
Copy link
Collaborator

@rock3r rock3r commented Jul 17, 2024

While it's nice to get ktlint warnings in the IDE, the ktlint IJ plugin shows an obnoxious editor banner when working on projects where it's not used.

Since we all work on other projects, and because their maintainers are not open to improving the UX, I don't recommend keeping this plugin anymore.

While it's nice to get ktlint warnings in the IDE, the ktlint IJ plugin
shows an obnoxious editor banner when working on projects where it's not
used.

Since we all work on other projects, and because their maintainers are
not open to improving the UX, I don't recommend keeping this plugin
anymore.
@rock3r rock3r added the maintenance Rote maintenance tasks (e.g., dependency updates) label Jul 17, 2024
@rock3r rock3r requested a review from hamen July 17, 2024 08:36
@rock3r rock3r self-assigned this Jul 17, 2024
@rock3r rock3r enabled auto-merge (squash) July 17, 2024 08:36
@rock3r rock3r merged commit e2f13ba into main Jul 17, 2024
4 checks passed
@rock3r rock3r deleted the no-ktlint-ij-plugin branch July 17, 2024 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Rote maintenance tasks (e.g., dependency updates)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants