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

Add support for nested SCMs #543

Merged
merged 4 commits into from
Dec 10, 2023
Merged

Conversation

jkloetzke
Copy link
Member

No description provided.

Add support to checkout an SCM into a subdirectory of another SCM. This
requires that the SCM with the upper directory is first in the list
before any other SCMs in subdirectories. Additionally, SCMs that are
natively supported by Jenkins plugins (git, svn), cannot be nested into
the other SCMs (cvs, import, url).  The reason is that Jenkins SCM
plugins always execute before anything else in a Jenkins job.

This relaxes the rules that did not allow any nesting before. There are
some drawbacks of such setups. Namely, if an SCM is moved to the attic,
all their nested SCMs are affected too.
Copy link

codecov bot commented Dec 10, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (cfd8e39) 88.33% compared to head (289a6e0) 88.38%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #543      +/-   ##
==========================================
+ Coverage   88.33%   88.38%   +0.05%     
==========================================
  Files          46       46              
  Lines       14511    14541      +30     
==========================================
+ Hits        12818    12852      +34     
+ Misses       1693     1689       -4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jkloetzke jkloetzke merged commit 0fec87a into BobBuildTool:master Dec 10, 2023
11 checks passed
@jkloetzke jkloetzke deleted the nested-scms branch December 10, 2023 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant