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

fix(jenkins) refactor (apply code conventions) plugin to fix changelog output #482

Merged
merged 4 commits into from
Jan 30, 2022

Conversation

dduportal
Copy link
Contributor

@dduportal dduportal commented Jan 24, 2022

Depends on #481 (need rebase once merged).

Fix #480.

This PR is a refactor of the jenkins plugin to allow continuing the work in #412.

It introduces the following changes:

Test

To test this pull request, you can run the following commands:

go build -o ./dist/updatecli && ./dist/updatecli diff --config examples/updatecli.generic/jenkins.yaml

Additional Information

N.A.

@dduportal dduportal force-pushed the chore/refactor-jenkins branch 4 times, most recently from 614670e to 8ffdb37 Compare January 29, 2022 10:00
@dduportal dduportal added bug Something isn't working cleanup condition Modify condition resource resource-jenkins Resource of kind Jenkins source labels Jan 29, 2022
@dduportal dduportal marked this pull request as ready for review January 29, 2022 10:05
@dduportal dduportal changed the title Wip: Chore/refactor jenkins fix(jenkins) refactor (apply code conventions) plugin to fix changelog output Jan 29, 2022
@dduportal dduportal force-pushed the chore/refactor-jenkins branch 2 times, most recently from 45951ed to 30aca7c Compare January 29, 2022 12:11
@dduportal
Copy link
Contributor Author

PR rebased since #481 has been merged.

For this PR, I'll wait for both your approval before merging, because it fixes a behavior :)

@dduportal dduportal force-pushed the chore/refactor-jenkins branch 2 times, most recently from fb7b268 to 97863e0 Compare January 30, 2022 09:39
@dduportal
Copy link
Contributor Author

Ping @olblak I would like to have this in 0.19.0

Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
… pages

Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
…code

Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
Copy link
Member

@olblak olblak left a comment

Choose a reason for hiding this comment

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

🥳

@olblak olblak merged commit 99485c1 into updatecli:main Jan 30, 2022
@dduportal dduportal deleted the chore/refactor-jenkins branch January 30, 2022 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cleanup condition Modify condition resource resource-jenkins Resource of kind Jenkins source
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Jenkins Sources do not retrieve changelog
2 participants