Skip to content

Commit 3e758b8

Browse files
authored
Maybe fix release push step (#4675)
The spec release is still failing to push the spec tests. Not sure why, but I think removing the `--depth=1` argument will fix it. It appears that it's referencing an object which it doesn't have. This used to work, not sure what changed. Maybe it had something to do with deterministic builds. I've also added a step to include the release tarballs in the consensus-specs release.
1 parent a668254 commit 3e758b8

File tree

1 file changed

+13
-1
lines changed

1 file changed

+13
-1
lines changed

.github/workflows/release.yml

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ jobs:
5454
- name: Clone spec tests repo
5555
run: |
5656
rm -rf consensus-spec-tests
57-
git clone https://x-access-token:${{ secrets.CONSENSUS_SPEC_TESTS_PAT }}@github.com/ethereum/consensus-spec-tests.git --branch=master --single-branch --depth=1 --no-tags
57+
git clone https://x-access-token:${{ secrets.CONSENSUS_SPEC_TESTS_PAT }}@github.com/ethereum/consensus-spec-tests.git --branch=master --single-branch --no-tags
5858
cd consensus-spec-tests
5959
rm -rf configs presets tests
6060
@@ -106,6 +106,18 @@ jobs:
106106
env:
107107
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
108108

109+
# Upload spec test tarballs to the specs release
110+
- name: Upload spec test tarballs
111+
uses: softprops/action-gh-release@6da8fa9354ddfdc4aeace5fc48d7f679b5214090 # v2.4.1
112+
with:
113+
tag_name: ${{ github.ref_name }}
114+
files: |
115+
consensus-spec-tests/general.tar.gz
116+
consensus-spec-tests/minimal.tar.gz
117+
consensus-spec-tests/mainnet.tar.gz
118+
env:
119+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
120+
109121
# Finally, publish the spec tests release
110122
# Requires a personal access token (PAT) with contents:read-write
111123
- name: Publish spec tests release

0 commit comments

Comments
 (0)