-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Issue #12311: checker-framework.groovy depends too much on name of repo #12337
Conversation
486b096
to
b165fc8
Compare
249b401
to
d9e42d8
Compare
Github CI does not like renaming of project directory (I tried to prove that update works in CI, see https://github.com/checkstyle/checkstyle/actions/runs/3305843693/jobs/5456285191). Here is proof from local:
|
@@ -1,28 +1,28 @@ | |||
<?xml version="1.0" encoding="UTF-8"?> | |||
<suppressedErrors> | |||
<checkerFrameworkError unstable="false"> | |||
<fileName>checkstyle/src/main/java/com/puppycrawl/tools/checkstyle/LocalizedMessage.java</fileName> |
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.
update of filename is required, since we do not include checkstyle
directory any longer.
d9e42d8
to
30c7993
Compare
30c7993
to
68d8f63
Compare
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.
worked for me on my local
Closes #12311