Skip to content

Add lint-changed task in Java workflow#3124

Closed
jagdish-15 wants to merge 1 commit intoexercism:mainfrom
jagdish-15:add-lint-changed
Closed

Add lint-changed task in Java workflow#3124
jagdish-15 wants to merge 1 commit intoexercism:mainfrom
jagdish-15:add-lint-changed

Conversation

@jagdish-15
Copy link
Copy Markdown
Member

Pull Request

We previously added test-changed to avoid the waiting time required to run tests on all exercises when only a few, or even just one, exercise was changed.

Even though linting took much less time than testing, there was still room to save around one minute per PR using this approach. I've tested it on my fork.

The working is analogous to the test-changed task, except this skips the result archive part.


Reviewer Resources:

Track Policies

@jagdish-15 jagdish-15 requested a review from kahgoh May 8, 2026 21:46
Comment thread bin/lint-changed-exercise
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This script seems quite similar to the build changed one in the other PR. Have you considered seeing if there is some way to share the common parts?

@jagdish-15
Copy link
Copy Markdown
Member Author

Superseded by #3126

@jagdish-15 jagdish-15 closed this May 9, 2026
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.

2 participants