Skip to content

ci.yml: avoid full git clone - #303

Open
oech3 wants to merge 1 commit into
uutils:mainfrom
oech3:patch-2
Open

oech3 wants to merge 1 commit into
uutils:mainfrom
oech3:patch-2

Conversation

@oech3

@oech3 oech3 commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

We don't need whole git histry. We should have --depth=1. However, tarball is more lightweight.

Comment thread .github/workflows/ci.yml
shell: bash
run: |
git clone https://github.com/mozilla/grcov.git ~/grcov/
mkdir -p ~/grcov

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

why not just git clone --depth=1? simpler and works :)

Comment thread .github/workflows/ci.yml
git clone https://github.com/mozilla/grcov.git ~/grcov/
mkdir -p ~/grcov
cd ~/grcov
curl -L https://github.com/mozilla/grcov/archive/refs/heads/master.tar.xz | tar -xzf -

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

.tar.xz with -z? this can't work, and I don't think github serves .tar.xz anyway
also the archive extracts into grcov-master/, so Cargo.toml isn't in ~/grcov
this is why the coverage job is red

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.

2 participants