Skip to content

Update MakeReleaseNotes to crash if notes are missing. #5613

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
Dec 21, 2023

Conversation

rlazo
Copy link
Collaborator

@rlazo rlazo commented Dec 21, 2023

The task is only invoked on projects that are scheduled to be released. And projects that publish refdocs are the ones that also publish release notes. The task now verifies that if it's called in a project that publishes refdocs, it has to have release notes, otherwise it breaks.

The task is only invoked on projects that are scheduled to be
released. And projects that publish refdocs are the ones that also
publish release notes. The task now verifies that if it's called in a
project that publishes refdocs, it has to have release notes,
otherwise it breaks.
@rlazo rlazo requested a review from daymxn December 21, 2023 18:18
Copy link
Contributor

github-actions bot commented Dec 21, 2023

📝 PRs merging into main branch

Our main branch should always be in a releasable state. If you are working on a larger change, or if you don't want this change to see the light of the day just yet, consider using a feature branch first, and only merge into the main branch when the code complete and ready to be released.

Copy link
Contributor

buildSrc Test Results

37 tests   37 ✔️  1m 2s ⏱️
  8 suites    0 💤
  8 files      0

Results for commit e5ded58.

@rlazo rlazo requested a review from davidmotson December 21, 2023 18:25
@rlazo rlazo enabled auto-merge (squash) December 21, 2023 18:26
Copy link
Contributor

Unit Test Results

     948 files  ±0       948 suites  ±0   31m 44s ⏱️ +23s
  5 178 tests ±0    5 157 ✔️ ±0  21 💤 ±0  0 ±0 
10 441 runs  ±0  10 399 ✔️ ±0  42 💤 ±0  0 ±0 

Results for commit e5ded58. ± Comparison against base commit 36cc041.

@google-oss-bot
Copy link
Contributor

@rlazo rlazo merged commit a5d0f0d into master Dec 21, 2023
@rlazo rlazo deleted the rl.missing.relnotes branch December 21, 2023 18:45
@google-oss-bot
Copy link
Contributor

Coverage Report 1

Affected Products

  • firebase-database

    Overall coverage changed from 50.13% (36cc041) to 50.14% (a5715f1) by +0.01%.

    FilenameBase (36cc041)Merge (a5715f1)Diff
    QueryParams.java89.76%90.36%+0.60%
  • firebase-firestore

    Overall coverage changed from 44.84% (36cc041) to 44.84% (a5715f1) by -0.00%.

    FilenameBase (36cc041)Merge (a5715f1)Diff
    SetMutation.java97.22%94.44%-2.78%
  • firebase-messaging

    Overall coverage changed from 84.00% (36cc041) to 83.89% (a5715f1) by -0.11%.

    FilenameBase (36cc041)Merge (a5715f1)Diff
    FirebaseMessaging.java73.57%73.13%-0.44%
    RequestDeduplicator.java90.00%80.00%-10.00%
  • firebase-storage

    Overall coverage changed from 84.08% (36cc041) to 83.92% (a5715f1) by -0.17%.

    FilenameBase (36cc041)Merge (a5715f1)Diff
    StorageException.java69.09%65.45%-3.64%
    StorageTask.java83.69%83.38%-0.30%
    UploadTask.java81.85%81.52%-0.33%

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/jQcuxTaUCi.html

@google-oss-bot
Copy link
Contributor

Startup Time Report 1

The report is too large (122,708 chars) to be displayed on GitHub. Please check this report on GCS.

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/Ar5ys5dok1/index.html

@firebase firebase locked and limited conversation to collaborators Jan 21, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants