Skip to content

Inplement ignorePaths and requiredPaths settings#152

Merged
danielflower merged 4 commits intodanielflower:masterfrom
nut-3:inplement-ignore-and-required-paths
May 24, 2025
Merged

Inplement ignorePaths and requiredPaths settings#152
danielflower merged 4 commits intodanielflower:masterfrom
nut-3:inplement-ignore-and-required-paths

Conversation

@nut-3
Copy link

@nut-3 nut-3 commented May 9, 2025

This PR addresses #77 and enables releases to be triggered only with certain file changes.

Copy link
Owner

@danielflower danielflower left a comment

Choose a reason for hiding this comment

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

Thanks - looks good overall. I only have a couple of small comments on the documentation. Get that done and I'll get it merged.


/**
* <p>List of file system paths that are required to be changed when detecting changes in the project(s).</p>
* <p>The primary purpose is to trigger creating new releases only f certain changes occur in the repository.
Copy link
Owner

Choose a reason for hiding this comment

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

Small typo: "...only f certain..."

I presume this is optional and if not set any file will cause it - can you spell this out at/near the top of this comment block?

Copy link
Author

Choose a reason for hiding this comment

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

Fixed typos and added clarification about the default behavior in the JavaDoc.

-----------------------------------------------------------

The primary purpose is to trigger creating new releases only f certain changes occur in the repository.
For example, a library repository with ProtoBuf messages would trigger release only if any of the *.proto files has been modified.
Copy link
Owner

Choose a reason for hiding this comment

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

Could you specify what wildcards are or aren't allowed - e.g. is it regex, glob patterns, or something else. And in the example exclude *.proto so it's obvious how to do that.

Copy link
Author

Choose a reason for hiding this comment

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

I've added clarification on how the plugin works if the option is not set and an explanation of the path matching logic. If something could be improved further, please give me a notice.

@nut-3 nut-3 force-pushed the inplement-ignore-and-required-paths branch from f23eeba to 931c90b Compare May 19, 2025 10:23
@danielflower danielflower merged commit fa34b6e into danielflower:master May 24, 2025
3 checks passed
@danielflower
Copy link
Owner

Thanks, this has been released in 3.8.0

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