Skip to content

Conversation

@martinbonnin
Copy link
Member

Forgetting to enable snapshots yields a 400 HTTP error. In those cases, give a hint to the user that the snapshots may be enabled in the UI.

@martinbonnin martinbonnin marked this pull request as draft July 20, 2025 20:32
@martinbonnin martinbonnin force-pushed the better-error-message branch from b548d07 to dbccea7 Compare August 9, 2025 21:09
@martinbonnin martinbonnin marked this pull request as ready for review August 9, 2025 21:09
@martinbonnin martinbonnin merged commit a76b0fa into main Aug 9, 2025
1 check passed
@martinbonnin martinbonnin deleted the better-error-message branch August 9, 2025 21:24
appendLine(" - Are your credentials correct?")
appendLine(" - Did you enable the snapshots on your namespace at https://central.sonatype.com/publishing/namespaces?")
// I have seen 400 for this
appendLine(" - Is your version ending with `-SNAPSHOT`?")
Copy link
Contributor

Choose a reason for hiding this comment

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

The plugin could check the version rather than guess

Copy link
Member Author

Choose a reason for hiding this comment

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

Right, I have added this to #82

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