Skip to content

Commit

Permalink
Merge pull request #2612 from eqlabs/t00ts/ci-remove-verify
Browse files Browse the repository at this point in the history
chore(ci): remove `verify-crates` job
  • Loading branch information
t00ts authored Feb 21, 2025
2 parents b843299 + e8b2c9f commit 7322afe
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 20 deletions.
10 changes: 5 additions & 5 deletions .github/workflows/publish-crates.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ jobs:
steps:
- uses: actions/checkout@v4
- uses: dtolnay/rust-toolchain@stable

- name: Login to crates.io
run: cargo login ${{ secrets.CRATES_IO_TOKEN }}

Expand All @@ -26,7 +26,7 @@ jobs:
local package=$1
local max_attempts=3
local attempt=1
while [ $attempt -le $max_attempts ]; do
echo "Attempting to publish $package (attempt $attempt/$max_attempts)"
if cargo publish -p $package; then
Expand All @@ -41,14 +41,14 @@ jobs:
# First publish crypto as it has no internal dependencies
publish_with_retry pathfinder-crypto
sleep 30
# Publish common which depends on crypto
publish_with_retry pathfinder-common
sleep 30
# Publish serde which depends on common and crypto
publish_with_retry pathfinder-serde
sleep 30
# Finally publish class-hash which depends on common, crypto and serde
publish_with_retry pathfinder-class-hash
15 changes: 0 additions & 15 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,21 +9,6 @@ on:
- v[0-9]+.*

jobs:
verify-crates:
name: Verify crates can be published
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: dtolnay/rust-toolchain@stable

# Verify each crate can be published
- name: Verify crates
run: |
for crate in crypto common serde class-hash; do
echo "Verifying pathfinder-$crate..."
cargo publish --dry-run -p pathfinder-$crate
done
create-draft-release:
runs-on: ubuntu-latest
steps:
Expand Down

0 comments on commit 7322afe

Please sign in to comment.