Skip to content

Prebuild Publishing Targets to Miinimize Chances of Releases Breaking#650

Merged
KetanReddy merged 3 commits intomainfrom
ops/build-resiliency
Jun 10, 2025
Merged

Prebuild Publishing Targets to Miinimize Chances of Releases Breaking#650
KetanReddy merged 3 commits intomainfrom
ops/build-resiliency

Conversation

@KetanReddy
Copy link
Copy Markdown
Member

@KetanReddy KetanReddy commented May 13, 2025

Change Type (required)

Indicate the type of change your pull request is:

  • patch
  • minor
  • major
  • N/A

Does your PR have any documentation updates?

  • Updated docs
  • No Update needed
  • Unable to update docs
📦 Published PR as canary version: 0.11.1--canary.650.22896

Try this version out locally by upgrading relevant packages to 0.11.1--canary.650.22896

@codecov
Copy link
Copy Markdown

codecov Bot commented May 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.38%. Comparing base (40e8a6e) to head (9c1e9a7).
Report is 32 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #650      +/-   ##
==========================================
- Coverage   90.39%   90.38%   -0.01%     
==========================================
  Files         333      333              
  Lines       20564    20564              
  Branches     1987     1987              
==========================================
- Hits        18588    18587       -1     
- Misses       1962     1963       +1     
  Partials       14       14              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 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.

@KetanReddy
Copy link
Copy Markdown
Member Author

/canary

Comment thread scripts/release.sh Outdated
MVN_RELEASE_TYPE=release
fi

# Android/JVM Prepublish
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can merge this comment w/ the Maven Central Prepublish comment — they're the same thing.

Comment thread scripts/release.sh Outdated
fi

# Android/JVM Prepublish
echo "Publishing Maven Packages with release type: ${MVN_RELEASE_TYPE} on branch: ${CURRENT_BRANCH}"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should move the echos until where we actually publish

Comment thread scripts/release.sh Outdated
# Running this here because it will still have the pre-release version in the VERSION file before auto cleans it up
# Make sure to re-stamp the outputs with the BASE_PATH so nextjs knows what to do with links
# Docs Prepublish
bazel build --config=release //docs/...
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should build the explicit target that we're going to run, //docs:gh_deploy — not everything under docs, as it might not all be applicable for publishing docs.

@KetanReddy KetanReddy added the skip-release Preserve the current version when merged label Jun 10, 2025
@KetanReddy KetanReddy merged commit ffcd46f into main Jun 10, 2025
2 of 4 checks passed
@KetanReddy KetanReddy deleted the ops/build-resiliency branch June 10, 2025 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-release Preserve the current version when merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants