Skip to content

Conversation

@danceratopz
Copy link
Member

@danceratopz danceratopz commented Apr 10, 2025

🗒️ Description

  1. Fixes the consume --input flag with non-release URLs:
    uv run consume direct --input "http://eest.fixtures/v0/fixtures_stable.tar.gz" --collect-only -q
    
  2. Fixes consume --input flag with "regular" numerical release specs <tag_name>@v1.2.3, (normal release specs, i.e., {stable,develop}@latest and pre-release specs, e.g., [email protected] were working):
    uv run consume direct --input [email protected] -k chainid --collect-only -q
    

🔗 Related Issues

Suggested alternative to

✅ Checklist

  • All: Set appropriate labels for the changes.
  • All: Considered squashing commits to improve commit history.
  • All: Added an entry to CHANGELOG.md.

@danceratopz danceratopz added type:bug Something isn't working scope:consume Scope: Consume command suite labels Apr 10, 2025
Copy link
Collaborator

@spencer-tb spencer-tb left a comment

Choose a reason for hiding this comment

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

All LGTM from my side! Thanks :)

Copy link
Member

@marioevz marioevz left a comment

Choose a reason for hiding this comment

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

Awesome! Thanks for implementing this, I'll go ahead and close #1435

@marioevz marioevz merged commit 23a354e into main Apr 11, 2025
21 checks passed
@marioevz marioevz deleted the fix-consume-input-with-non-release-url branch April 11, 2025 19:17
pacrob pushed a commit to pacrob/execution-spec-tests that referenced this pull request May 5, 2025
…l release specs (ethereum#1437)

* fix(consume): fix downloading non-release url tarballs

* fix(consume): fix downloading via release spec with numerical version

* docs: update changelog

* fix typo
felix314159 pushed a commit to felix314159/execution-spec-tests that referenced this pull request May 16, 2025
…l release specs (ethereum#1437)

* fix(consume): fix downloading non-release url tarballs

* fix(consume): fix downloading via release spec with numerical version

* docs: update changelog

* fix typo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope:consume Scope: Consume command suite type:bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants