Skip to content

Do not upload placeholders to S3#3107

Merged
ildyria merged 1 commit intomasterfrom
do-not-upload-placeholders
Mar 19, 2025
Merged

Do not upload placeholders to S3#3107
ildyria merged 1 commit intomasterfrom
do-not-upload-placeholders

Conversation

@ildyria
Copy link
Member

@ildyria ildyria commented Mar 19, 2025

Fixes #3108

Only fixing the active path.

Not sure if legacy is impacted. Don't care. 😆

@ildyria ildyria requested a review from a team as a code owner March 19, 2025 16:29
@ildyria ildyria added the Review: easy Easy review expected: probably just need a quick to go through. label Mar 19, 2025
@codecov
Copy link

codecov bot commented Mar 19, 2025

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 90.44%. Comparing base (35f2e24) to head (013d52e).
Report is 2 commits behind head on master.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ildyria ildyria merged commit 9d4b073 into master Mar 19, 2025
54 checks passed
@ildyria ildyria deleted the do-not-upload-placeholders branch March 19, 2025 22:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Review: easy Easy review expected: probably just need a quick to go through.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants