Skip to content

Conversation

@NaddiNadja
Copy link
Contributor

@NaddiNadja NaddiNadja commented Dec 18, 2025

This solves issue #198

I've added it as both an argument in the configuration file, so that you can adjust these args per repository, or in the workflow file, so as to let you just adjust it for every repository. The configuration file will overwrite the setting in the workflow.

In the configuration file:

[cijoe.repository]
remote = "https://github.com/refenv/cijoe/"
path = "{{ local.env.HOME }}/git/cijoe"
depth = 2
single_branch = true

In the workflow:

- name: repos
  uses: core.repository_prep
  with:
    depth: 2
    single_branch: true

This solves issue refenv#198

Signed-off-by: Nadja Brix Koch <n.koch@samsung.com>
Copy link
Collaborator

@safl safl left a comment

Choose a reason for hiding this comment

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

Lgtm

@safl safl merged commit 75d66d2 into refenv:main Dec 18, 2025
59 of 60 checks passed
@NaddiNadja NaddiNadja deleted the repo branch December 18, 2025 14:05
@coveralls
Copy link

Pull Request Test Coverage Report for Build 20332519363

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 11 of 13 (84.62%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.06%) to 78.482%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/cijoe/core/scripts/repository_prep.py 11 13 84.62%
Totals Coverage Status
Change from base Build 18189340355: 0.06%
Covered Lines: 1592
Relevant Lines: 2094

💛 - Coveralls


err, _ = run(
f"[ ! -d {repos['path']} ] &&"
f" git clone {repos['remote']} {repos['path']} --recursive"
Copy link

Choose a reason for hiding this comment

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

maybe --recursive should also become an argument...

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.

4 participants