Skip to content

Conversation

@DavideD
Copy link
Member

@DavideD DavideD commented Aug 27, 2024

Part of #1956

  • remove the JBoss Nexus repository from the configuration
  • remove release jobs from the GitHub workflow
  • add a first draft of the Jenkins pipeline

@yrodiere Maybe you want to have a quick look? It mimics the one pipeline in ORM, but I'm not updating the website and not using the JIRA. Basically, I've tried to keep the changes to the existing release task to a minimum.
I will improve everything when we have tested that it actually works with a real release.

@DavideD DavideD force-pushed the 1956-release-update branch from ace71b3 to 4a7c515 Compare August 27, 2024 13:26
@DavideD DavideD merged commit 4a7c515 into hibernate:main Aug 28, 2024
Copy link
Member

@yrodiere yrodiere left a comment

Choose a reason for hiding this comment

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

Again sorry for the delay. I had nothing important to mention here though.

name: reports-java${{ matrix.java.name }}
path: './**/build/reports/'

snapshot:
Copy link
Member

Choose a reason for hiding this comment

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

Do you plan to restore snapshot publishing somehow?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, I'm working on it now.


env.RELEASE_VERSION = releaseVersion.toString()
env.DEVELOPMENT_VERSION = developmentVersion.toString()
// Dry run is not supported at the moment
Copy link
Member

Choose a reason for hiding this comment

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

You mean to add this in a later PR?

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah, this is just a leftover. Dry run is working

@DavideD
Copy link
Member Author

DavideD commented Aug 28, 2024

Again sorry for the delay. I had nothing important to mention here though

No problem. In the end I decided to merge it and apply changes later if needed.

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