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

[packagers] Brew packager is silently skipped if distribution file names do not end up with zip or jar #1033

Closed
maciejwalkowiak opened this issue Nov 8, 2022 · 5 comments
Assignees
Labels
ant Affects Ant tasks cli Affects CLI enhancement New feature or request gradle Affects Gradle plugins maven Affects Maven plugins released Issue has been released
Milestone

Comments

@maciejwalkowiak
Copy link

When artifacts files do not end up with zip or jar (as described as required in docs), publishing to Homebrew is just skipped without any error or warning message.

I believe there should be a warning explaining why brew packaging is skipped, or perhaps the whole release should fail.

@maciejwalkowiak maciejwalkowiak added the bug Something isn't working label Nov 8, 2022
@aalmiray aalmiray changed the title Brew packager is silently skipped if distribution file names do not end up with zip or jar [packagers] Brew packager is silently skipped if distribution file names do not end up with zip or jar Nov 8, 2022
@aalmiray
Copy link
Member

aalmiray commented Nov 8, 2022

The trace.log (also when running the tool in with -g or --debug) prints out the following message when there are no matching artifacts (during validation):

- disabled because there were no matching artifacts

This message could be upgraded to a warning.
Automatically turning this warning into an error would result in incompatible behavior thus should be reserved for 2.x. However, a strict mode could be added to JReleaser that results in warnings turned into errors. The strict mode would have to be explicitly enabled. In 2.x strict could become the default. WDYT?

@maciejwalkowiak
Copy link
Author

To me this message is not self explanatory. ".. no matching artifacts" and then I think it would make sense to say conditions must be met that the artifact is matched, or what is wrong with current artifacts.

I think strict mode sound like a good idea - it should fail if there is any configuration mistake

@aalmiray
Copy link
Member

aalmiray commented Nov 8, 2022

At first I thought checking if strict mode is enabled by looking up a System property such as jreleaser.strict. However, that would set precedent for other settings that are already specified using command flags (CLI, Gradle) or goal/task properties (Maven, Gradle, Ant).

I suppose adding --strict (CLI, Gradle), jreleaser.strict (Maven, Gradle, Ant), and <strict> (Maven) would be consistent with other settings such as dry-run.

@aalmiray aalmiray self-assigned this Nov 9, 2022
@aalmiray aalmiray added this to the v1.4.0 milestone Nov 9, 2022
aalmiray added a commit to jreleaser/jreleaser.github.io that referenced this issue Nov 9, 2022
@maciejwalkowiak
Copy link
Author

🚀

@aalmiray aalmiray added ant Affects Ant tasks cli Affects CLI gradle Affects Gradle plugins maven Affects Maven plugins enhancement New feature or request and removed bug Something isn't working labels Nov 9, 2022
aalmiray added a commit to jreleaser/jreleaser.github.io that referenced this issue Nov 10, 2022
aalmiray added a commit to jreleaser/jreleaser.github.io that referenced this issue Nov 10, 2022
aalmiray added a commit to jreleaser/jreleaser.github.io that referenced this issue Nov 10, 2022
aalmiray added a commit to jreleaser/jreleaser.github.io that referenced this issue Dec 2, 2022
aalmiray added a commit to jreleaser/jreleaser.github.io that referenced this issue Dec 13, 2022
@aalmiray aalmiray added the released Issue has been released label Dec 29, 2022
@aalmiray
Copy link
Member

Released in v1.4.0 -> https://github.com/jreleaser/jreleaser/releases/tag/v1.4.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ant Affects Ant tasks cli Affects CLI enhancement New feature or request gradle Affects Gradle plugins maven Affects Maven plugins released Issue has been released
Projects
None yet
Development

No branches or pull requests

2 participants