Skip to content

Better symbol for GitSCMSource#1813

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

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

Conversation

@jglick
Copy link
Member

@jglick jglick commented Sep 30, 2025

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

Copy link
Contributor

@MarkEWaite MarkEWaite left a comment

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