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

Remove use of Maven Wagon #499

Merged
merged 1 commit into from
Jun 7, 2023
Merged

Remove use of Maven Wagon #499

merged 1 commit into from
Jun 7, 2023

Conversation

basil
Copy link
Member

@basil basil commented Jun 7, 2023

Per this page Maven Wagon is deprecated. This PR eliminates use of Maven Wagon at release time, replacing it with a GitHub Action that uploads the generated site to GitHub Pages on every clean build of the main branch.

Testing done

I am using this workflow successfully in Job DSL plugin. If there are any issues with this PR I will fix them after merging this PR.

Fixes #492

Comment on lines -328 to -332
<dependency>
<groupId>com.github.stephenc.wagon</groupId>
<artifactId>wagon-gitsite</artifactId>
<version>0.5</version>
</dependency>
Copy link
Member Author

Choose a reason for hiding this comment

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

https://github.com/stephenc/wagon-gitsite has not been updated since 2014, so this is a dead end.

@basil basil merged commit fd678c9 into jenkinsci:master Jun 7, 2023
5 checks passed
@basil
Copy link
Member Author

basil commented Jun 7, 2023

Deployed successfully to http://jenkinsci.github.io/maven-hpi-plugin/

@basil
Copy link
Member Author

basil commented Jun 7, 2023

Hrmm, the generated site seems to be missing the Mojo pages. But I see the same problem when running mvn clean verify site before this PR. Need to figure out why generating the site during the release generates the Mojo pages but generating the site outside of a release doesn't.

@basil basil mentioned this pull request Jun 7, 2023
@basil
Copy link
Member Author

basil commented Jun 7, 2023

--> #500

@basil
Copy link
Member Author

basil commented Jun 7, 2023

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.

Remove use of Maven Wagon
2 participants