Skip to content

test: Avoid publishing old tarballs for E2E tests #16784

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 2, 2025

Conversation

mydea
Copy link
Member

@mydea mydea commented Jul 1, 2025

I noticed that locally I sometimes have old tarballs lying around, which are all published for E2E tests. This is unnecessary, and can also make it a bit unreliable which version is used for tests (as we generally have the dependencies as * which could be anything). This changes it so we only publish the current version.

I noticed that locally I sometimes have old tarballs lying around, which are all published for E2E tests. This is unnecessary, and can also make it a bit unreliable which version is used for tests (as we generally have the dependencies as `*` which could be anything). This changes it so we only publish the current version.
@mydea mydea requested review from s1gr1d and andreiborza July 1, 2025 13:10
@mydea mydea self-assigned this Jul 1, 2025
Copy link

cursor bot commented Jul 1, 2025

🚨 BugBot couldn't run

Something went wrong. Try again by commenting "bugbot run", or contact support (requestId: serverGenReqId_0da9d9f1-3339-4f76-a0c3-5ba8f9aa0772).

Copy link
Member

@s1gr1d s1gr1d left a comment

Choose a reason for hiding this comment

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

Nice!

@mydea mydea merged commit 3913754 into develop Jul 2, 2025
144 of 145 checks passed
@mydea mydea deleted the fn/avoid-e2e-old-tarballs branch July 2, 2025 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants