Skip to content

Commit

Permalink
gen: remove pointless "update"
Browse files Browse the repository at this point in the history
libbpf is a submodule, so its version is already effectively pinned.
`xtask codegen` also runs `git submodule update` which reverts the
action of updating to `origin/HEAD`. Remove the cruft.
  • Loading branch information
tamird committed Feb 5, 2025
1 parent 8859b76 commit 6baa81e
Showing 1 changed file with 3 additions and 8 deletions.
11 changes: 3 additions & 8 deletions .github/workflows/gen.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,14 +11,6 @@ jobs:
with:
submodules: recursive

- name: update libbpf
working-directory: xtask/libbpf
run: |
set -euxo pipefail
git fetch origin
git checkout origin/HEAD
echo "LIBBPF_SHA=$(git rev-parse HEAD)" >> $GITHUB_ENV
- uses: dtolnay/rust-toolchain@nightly
with:
components: rustfmt, clippy
Expand All @@ -34,6 +26,9 @@ jobs:
- run: cargo xtask codegen
- run: cargo xtask public-api --bless

- run: echo "LIBBPF_SHA=$(git rev-parse HEAD)" >> $GITHUB_ENV
working-directory: xtask/libbpf

- uses: peter-evans/create-pull-request@v7
with:
branch: create-pull-request/codegen
Expand Down

0 comments on commit 6baa81e

Please sign in to comment.