Skip to content

Better symbol for GitSCMSource - #1813

Merged
MarkEWaite merged 1 commit into
jenkinsci:masterfrom
jglick:gitSource
Oct 3, 2025
Merged

Better symbol for GitSCMSource#1813
MarkEWaite merged 1 commit into
jenkinsci:masterfrom
jglick:gitSource

Conversation

@jglick

@jglick jglick commented Sep 30, 2025

Copy link
Copy Markdown
Member

d1ee85b introduced a symbol but it is unusable from Pipeline script, as noted in a similar case in #1373.

@MarkEWaite MarkEWaite left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Change looks good to me. Thanks!

@MarkEWaite
MarkEWaite merged commit 785dcce into jenkinsci:master Oct 3, 2025
18 checks passed
@jglick
jglick deleted the gitSource branch October 6, 2025 13:10
@MarkEWaite MarkEWaite added the enhancement Improvement or new feature label Oct 8, 2025
MarkEWaite added a commit to MarkEWaite/pipeline-groovy-lib-plugin that referenced this pull request Oct 9, 2025
Git plugin 5.8.0 provides the gitSource symbol so that retrievers can
replace the $class syntax with the gitSource symbol.  Refer to git plugin
pull request:

* jenkinsci/git-plugin#1813

Without this fix, plugin BOM tests fail.

Updates the plugin dependencies to match the dependencies needed by
git plugin 5.8.0.  Updates include comments to remind that they can be
removed once git plugin 5.8.0 is in the plugin BOM.

Testing done:

* Confirmed that plugin BOM fails without this change:

  LINE=weekly PLUGINS=pipeline-groovy-lib TEST=LibraryStepTest#configRoundtrip bash ./local-test.sh

* Confirmed that plugin tests pass with this change

  mvn clean verify
MarkEWaite added a commit to MarkEWaite/docker-lfs that referenced this pull request Oct 11, 2025
jglick added a commit to jglick/workflow-scm-step-plugin that referenced this pull request Oct 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Improvement or new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants