Skip to content

Conversation

kwin
Copy link
Contributor

@kwin kwin commented Oct 10, 2025

Otherwise people not having according settings.xml get misleading error messages during release. Compare with maveniverse/njord#162 (comment)

Comment on lines +361 to +362
<njord.releaseUrl>njord:template:release-sca</njord.releaseUrl>
<njord.publisher>sonatype-cp</njord.publisher>
Copy link
Member

Choose a reason for hiding this comment

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

We have the same in local settings.xml .... so should be dropped from settings.xml

https://codehaus-plexus.github.io/plexus-pom/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point, fixed doc in 7466122.

@kwin kwin force-pushed the njord-properties branch from aefed00 to 7466122 Compare October 10, 2025 16:04
Otherwise people not having according `settings.xml` get misleading
error messages during release. Compare with
maveniverse/njord#162 (comment)
@kwin kwin force-pushed the njord-properties branch from 7466122 to 37861b2 Compare October 10, 2025 16:06
@slawekjaranowski slawekjaranowski merged commit b40174c into master Oct 11, 2025
14 checks passed
@slawekjaranowski slawekjaranowski deleted the njord-properties branch October 11, 2025 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants