Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add tests to make sure rules have tests #349

Merged
merged 1 commit into from
Oct 4, 2024

Conversation

mrmans0n
Copy link
Owner

@mrmans0n mrmans0n commented Oct 4, 2024

This is to make sure external contributors don't forget to add the detekt/ktlint version of the rules (I've had some instances of this, people typically forget to include the ktlint version), and that all rules in both detekt/ktlint have unit tests.

This would make working in this repo more straightforward, as tools would yell at you if you forget something. It's a lot, and it's getting hard to track, I know 😆

@mrmans0n mrmans0n merged commit b942ecf into main Oct 4, 2024
2 checks passed
@mrmans0n mrmans0n deleted the nacho/ensure-rules-have-tests branch October 4, 2024 09:49
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant