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 the ability to configure ignoredFiles #236

Merged
merged 2 commits into from
Feb 13, 2024

Conversation

kpagratis
Copy link
Contributor

It would be useful to have the ability to configure files to be ignored by the plugin.
With this functionality, you could filter files like markdown docs so your pipelines can run faster.

@CLAassistant
Copy link

CLAassistant commented Nov 3, 2023

CLA assistant check
All committers have signed the CLA.

@joshafeinberg
Copy link
Member

@kpagratis thanks for the contribution, can you please run ktlint locally to get some fixes in for your changes then I can probably get this in for the next release :)

@kpagratis
Copy link
Contributor Author

@joshafeinberg Yes of course. I'll do that tomorrow. Thanks for taking a look!

@kpagratis
Copy link
Contributor Author

❯ ./gradlew ktlintCheck

Deprecated Gradle features were used in this build, making it incompatible with Gradle 8.0.

You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.

See https://docs.gradle.org/7.4.2/userguide/command_line_interface.html#sec:command_line_warnings

BUILD SUCCESSFUL in 339ms
1 actionable task: 1 up-to-date
❯ ./gradlew ktlintFormat

Deprecated Gradle features were used in this build, making it incompatible with Gradle 8.0.

You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.

See https://docs.gradle.org/7.4.2/userguide/command_line_interface.html#sec:command_line_warnings

BUILD SUCCESSFUL in 324ms
1 actionable task: 1 up-to-date

@joshafeinberg
Copy link
Member

Whoops, looks we applied ktlint wrong so i'll merge this and then fix that up. Thanks again

@joshafeinberg joshafeinberg merged commit ae16557 into dropbox:main Feb 13, 2024
1 check passed
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.

3 participants