-
Notifications
You must be signed in to change notification settings - Fork 861
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
Ignore .idea directory in spotless checks #4657
Merged
daniellehrner
merged 3 commits into
hyperledger:main
from
daniellehrner:spotless_ignore_.idea
Nov 14, 2022
Merged
Ignore .idea directory in spotless checks #4657
daniellehrner
merged 3 commits into
hyperledger:main
from
daniellehrner:spotless_ignore_.idea
Nov 14, 2022
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>
fab-10
approved these changes
Nov 11, 2022
shemnon
suggested changes
Nov 11, 2022
build.gradle
Outdated
Comment on lines
131
to
132
target '**/*.java' | ||
targetExclude '**/src/reference-test/**', '**/src/main/generated/**', '**/src/test/generated/**', '**/src/jmh/generated/**' | ||
targetExclude '**/src/reference-test/**', '**/src/main/generated/**', '**/src/test/generated/**', '**/src/jmh/generated/**', '.idea/**' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rather than one-by-one exclusions we should have a more focused inclusion.
Suggested change
target '**/*.java' | |
targetExclude '**/src/reference-test/**', '**/src/main/generated/**', '**/src/test/generated/**', '**/src/jmh/generated/**' | |
targetExclude '**/src/reference-test/**', '**/src/main/generated/**', '**/src/test/generated/**', '**/src/jmh/generated/**', '.idea/**' | |
target 'src/**/*.java' | |
targetExclude 'src/reference-test/**', 'src/main/generated/**', 'src/test/generated/**', 'src/jmh/generated/**' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are right. That is much better. I've changed it.
…sion. Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>
shemnon
approved these changes
Nov 14, 2022
wcgcyx
pushed a commit
to wcgcyx/besu
that referenced
this pull request
Nov 16, 2022
Signed-off-by: wcgcyx <wcgcyx@gmail.com>
macfarla
pushed a commit
to macfarla/besu
that referenced
this pull request
Jan 10, 2023
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
eum602
pushed a commit
to lacchain/besu
that referenced
this pull request
Nov 3, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Daniel Lehrner daniel.lehrner@consensys.net
PR description
InteliJ allows to define templates for different files like Java classes, interfaces,... If those templates are saved in the project they are saved as
.java
files in a subdirectory of.idea
.Such a template looks for example like this:
Spotless currently tries to check this templates, because it is configured to analyze all Java files. This check will fail, because the templates are not valid Java code.
This PR excludes all the files in the
.idea
directory from the spotless check to avoid those false positives.Documentation
doc-change-required
label to this PR ifupdates are required.
Changelog