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

Update workflow-cps, antisamy-markup-formatter, and the PCT #1553

Merged
merged 5 commits into from
Nov 4, 2022

Conversation

dwnusbaum
Copy link
Member

@dwnusbaum dwnusbaum commented Nov 3, 2022

Originally filed to test jenkinsci/workflow-cps-plugin#612 and jenkinsci/plugin-compat-tester#383, but now it updates everything related to those PRs.

@dwnusbaum
Copy link
Member Author

Looks good in general. The only issue so far is with antisamy-markup-formatter because jenkinsci/antisamy-markup-formatter-plugin#65 has not yet been released and so it does not support incrementals:

[2022-11-03T19:50:29.619Z] [ERROR] Failed to execute goal org.jenkins-ci.tools:maven-hpi-plugin:3.33:resolve-test-dependencies (default-resolve-test-dependencies) on project antisamy-markup-formatter: Unable to resolve dependencies: Could not resolve dependencies for project org.jenkins-ci.plugins:antisamy-markup-formatter:hpi:2.7: Could not find artifact org.jenkins-ci.plugins.workflow:workflow-cps:jar:3513.v1b_01482f7077 in repo.jenkins-ci.org (https://repo.jenkins-ci.org/public/) -> [Help 1]

workflow-cps itself has passed against the latest weekly, but hasn't run against 2.332.x yet as far as I can tell.

@dwnusbaum
Copy link
Member Author

I will push a commit to set failFast to false so we know for sure whether antisamy-markup-formatter is the only problem.

@jglick
Copy link
Member

jglick commented Nov 3, 2022

has not yet been released

Hmm, I will fix that.

@jglick
Copy link
Member

jglick commented Nov 3, 2022

@dwnusbaum
Copy link
Member Author

The GitHub check mentioning that caffeine-api failed is just a reporting bug. The only plugin that failed is indeed antisamy-markup-formatter in the weekly branch. I will update it and remove failFast = false and then I think all of the downstream stuff is ready to be merged.

prep.sh Outdated Show resolved Hide resolved
@dwnusbaum dwnusbaum changed the title Test changes related to converting workflow-cps into a multi-module project Update workflow-cps, antisamy-markup-formatter, and the PCT Nov 4, 2022
bom-weekly/pom.xml Outdated Show resolved Hide resolved
@dwnusbaum dwnusbaum marked this pull request as ready for review November 4, 2022 22:09
@jglick jglick added the dependencies Pull requests that update a dependency file label Nov 4, 2022
@jglick jglick enabled auto-merge (squash) November 4, 2022 22:32
@jglick jglick merged commit 3066aa1 into jenkinsci:master Nov 4, 2022
@dwnusbaum dwnusbaum deleted the workflow-cps-multi-module branch November 5, 2022 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants