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.

Sign up for GitHub

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

Jump to bottom

Added script to update the Jenkins plugins #24

Merged
merged 2 commits into from
May 6, 2020
Merged

Added script to update the Jenkins plugins #24

merged 2 commits into from
May 6, 2020
+233 −4

Conversation

alxsap
Copy link
Contributor

@alxsap alxsap commented May 5, 2020

I added the script we use to update the plugins.
The whole process should be automated.
Missing: Update Jenkins to LTS via script.

@alxsap
Added script to update the Jenkins plugins
38582c4 
I added the script we use to update the plugins.
The whole process should be automated.
Missing: Update Jenkins to LTS via script.
@alxsap alxsap mentioned this pull request May 5, 2020
Closed
alxsap added a commit that referenced this pull request May 5, 2020
@alxsap
Removing parts which moved to PR #24
aeae9f0
romanisb
romanisb reviewed May 6, 2020
View reviewed changes
update/updatePlugins.sh Outdated Show resolved Hide resolved
romanisb
romanisb approved these changes May 6, 2020
View reviewed changes
Copy link
Contributor

@romanisb romanisb left a comment

Choose a reason for hiding this comment

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

Looks good. At one point the generate.groovy script needs to take the compatibility of a plugin with the used Jenkins core into account.

@alxsap
Copy link
Contributor Author

alxsap commented May 6, 2020

Looks good. At one point the generate.groovy script needs to take the compatibility of a plugin with the used Jenkins core into account.

Created #25

@alxsap alxsap merged commit 50f6d12 into master May 6, 2020
@alxsap alxsap deleted the PR_pluginUpdateScript branch May 6, 2020 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@romanisb romanisb romanisb approved these changes

@rinckm rinckm Awaiting requested review from rinckm

@holgpar holgpar Awaiting requested review from holgpar

@anbrsap anbrsap Awaiting requested review from anbrsap

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@alxsap @romanisb