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

Setup CD #64

Merged
merged 1 commit into from
Dec 31, 2021
Merged

Setup CD #64

merged 1 commit into from
Dec 31, 2021

Conversation

NotMyFault
Copy link
Member

@guysoft Make sure you have the following secrets at https://github.com/jenkinsci/configurationslicing-plugin/settings/secrets/actions before merging. But considering the RPU PR has been merged, it should all be set up.

Make sure to put an "interesting label" like feature or enhancement on this and the other PR so CD can pick it up.

I've addressed the inconsistency of tabs and spaces in the pom as well.

@guysoft
Copy link
Contributor

guysoft commented Dec 31, 2021

Checked and its there. Thanks for all the help"

@guysoft guysoft merged commit da32b46 into jenkinsci:master Dec 31, 2021
@NotMyFault NotMyFault deleted the feat/master/cd branch December 31, 2021 16:56
@NotMyFault
Copy link
Member Author

You forgot the labels of the PRs 👀

@guysoft
Copy link
Contributor

guysoft commented Dec 31, 2021

You mean this one or all current ones?

@NotMyFault
Copy link
Member Author

NotMyFault commented Dec 31, 2021

I meant this and my other one. Release drafter does only do releases if content is merged that may be interesting for the user.
otherwise you would always have new releases for dependency updates e.g.
But you can always trigger a release by hand if you request to run the workflow manual again.

@guysoft guysoft added the feature label Jan 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants