Skip to content

Commit

Permalink
ci: simplify the codegen workflow a bit
Browse files Browse the repository at this point in the history
the codegen ci workflow was previously fixed by copying the checkout dir
to the GOPATH. With the updated codegen tooling, this alignment is not a
requirement any more.

This simplifies the codegen workflow again by reverting the earlier
changes that added the copy mechanism.

This reverts commit 714f80a.
This reverts commit f32d3df.

Signed-off-by: Michael Adam <[email protected]>
  • Loading branch information
obnoxxx committed Dec 5, 2024
1 parent fe9664f commit 46fae52
Showing 1 changed file with 2 additions and 12 deletions.
14 changes: 2 additions & 12 deletions .github/workflows/codegen.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,18 +38,8 @@ jobs:
with:
go-version: "1.22"

- name: post-process checkout - copy to github.com
run: |
# copy to the github.com path so codegen will work as expected
mkdir -p $(go env GOPATH)/src/github.com/rook/rook/
cp -R . $(go env GOPATH)/src/github.com/rook/rook/
- name: run codegen
run: |
cd $(go env GOPATH)/src/github.com/rook/rook
GOPATH=$(go env GOPATH) make codegen
run: GOPATH=$(go env GOPATH) make codegen

- name: validate codegen
run: |
cd $(go env GOPATH)/src/github.com/rook/rook
tests/scripts/validate_modified_files.sh codegen
run: tests/scripts/validate_modified_files.sh codegen

0 comments on commit 46fae52

Please sign in to comment.