Skip to content

Add a workflow to merge release into master #6217

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 2 commits into from
May 4, 2022
Merged

Conversation

hsubox76
Copy link
Contributor

@hsubox76 hsubox76 commented May 3, 2022

Manually triggerable workflow that merges release into master. Won't be able to test it until after next release.

@hsubox76 hsubox76 requested a review from allspain as a code owner May 3, 2022 22:55
@changeset-bot
Copy link

changeset-bot bot commented May 3, 2022

⚠️ No Changeset found

Latest commit: d7c2485

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@hsubox76 hsubox76 requested a review from lahirumaramba May 3, 2022 22:55
@google-oss-bot
Copy link
Contributor

google-oss-bot commented May 3, 2022

Size Report 1

Affected Products

  • @firebase/app

    TypeBase (5c92d9c)Merge (acb4fe5)Diff
    browser13.8 kB13.8 kB+21 B (+0.2%)
    esm518.0 kB18.0 kB+21 B (+0.1%)
    main18.9 kB19.0 kB+50 B (+0.3%)
    module13.8 kB13.8 kB+21 B (+0.2%)
  • @firebase/installations

    TypeBase (5c92d9c)Merge (acb4fe5)Diff
    browser17.8 kB17.8 kB+13 B (+0.1%)
    esm522.9 kB22.9 kB+13 B (+0.1%)
    main23.7 kB23.8 kB+42 B (+0.2%)
    module17.8 kB17.8 kB+13 B (+0.1%)
  • @firebase/messaging

    TypeBase (5c92d9c)Merge (acb4fe5)Diff
    browser20.8 kB20.8 kB+25 B (+0.1%)
    esm526.2 kB26.2 kB+25 B (+0.1%)
    main26.8 kB26.9 kB+50 B (+0.2%)
    module20.8 kB20.8 kB+25 B (+0.1%)
  • @firebase/messaging-sw

    TypeBase (5c92d9c)Merge (acb4fe5)Diff
    main29.6 kB29.6 kB+50 B (+0.2%)
    module22.8 kB22.9 kB+25 B (+0.1%)
  • @firebase/util

    TypeBase (5c92d9c)Merge (acb4fe5)Diff
    browser23.4 kB20.5 kB-2.87 kB (-12.3%)
    esm525.5 kB21.9 kB-3.68 kB (-14.4%)
    main30.6 kB26.7 kB-3.86 kB (-12.6%)
    module23.4 kB20.5 kB-2.87 kB (-12.3%)
  • bundle

    43 size changes

    TypeBase (5c92d9c)Merge (acb4fe5)Diff
    analytics (logEvent)40.0 kB41.5 kB+1.50 kB (+3.8%)
    app-check (CustomProvider)33.7 kB35.2 kB+1.51 kB (+4.5%)
    app-check (ReCaptchaEnterpriseProvider)35.9 kB37.4 kB+1.51 kB (+4.2%)
    app-check (ReCaptchaV3Provider)35.8 kB37.4 kB+1.51 kB (+4.2%)
    auth (Anonymous)63.5 kB65.0 kB+1.52 kB (+2.4%)
    auth (EmailAndPassword)67.6 kB69.1 kB+1.52 kB (+2.2%)
    auth (GoogleFBTwitterGitHubPopup)87.4 kB88.9 kB+1.52 kB (+1.7%)
    auth (GooglePopup)87.1 kB88.6 kB+1.52 kB (+1.7%)
    auth (GoogleRedirect)87.3 kB88.8 kB+1.52 kB (+1.7%)
    auth (Phone)73.6 kB75.1 kB+1.52 kB (+2.1%)
    database (Append to a list of data)144 kB145 kB+1.52 kB (+1.1%)
    database (Filtering data)142 kB144 kB+1.52 kB (+1.1%)
    database (Listen for child events)158 kB160 kB+1.52 kB (+1.0%)
    database (Listen for value events + Detach listeners)158 kB160 kB+1.52 kB (+1.0%)
    database (Listen for value events)158 kB160 kB+1.52 kB (+1.0%)
    database (Read data once)150 kB151 kB+1.52 kB (+1.0%)
    database (Save data as transactions)160 kB162 kB+1.52 kB (+0.9%)
    database (Sort data)144 kB146 kB+1.52 kB (+1.1%)
    database (Write data)143 kB144 kB+1.52 kB (+1.1%)
    firestore (Persistence)263 kB265 kB+1.51 kB (+0.6%)
    firestore (Query Cursors)203 kB204 kB+1.51 kB (+0.7%)
    firestore (Query)204 kB205 kB+1.51 kB (+0.7%)
    firestore (Read data once)192 kB194 kB+1.51 kB (+0.8%)
    firestore (Realtime updates)195 kB196 kB+1.51 kB (+0.8%)
    firestore (Transaction)177 kB178 kB+1.51 kB (+0.9%)
    firestore (Write data)176 kB177 kB+1.51 kB (+0.9%)
    firestore-lite (Query Cursors)66.3 kB67.8 kB+1.50 kB (+2.3%)
    firestore-lite (Query)69.5 kB71.0 kB+1.50 kB (+2.2%)
    firestore-lite (Read data once)53.9 kB55.4 kB+1.50 kB (+2.8%)
    firestore-lite (Transaction)71.2 kB72.7 kB+1.50 kB (+2.1%)
    firestore-lite (Write data)56.6 kB58.1 kB+1.50 kB (+2.7%)
    functions (call)27.6 kB29.1 kB+1.50 kB (+5.4%)
    messaging (send + receive)43.3 kB44.7 kB+1.34 kB (+3.1%)
    performance (trace)47.7 kB49.2 kB+1.50 kB (+3.1%)
    remote-config (getAndFetch)42.4 kB43.9 kB+1.50 kB (+3.5%)
    storage (getBytes)35.9 kB37.4 kB+1.50 kB (+4.2%)
    storage (getDownloadURL)38.0 kB39.5 kB+1.50 kB (+4.0%)
    storage (getMetadata)37.5 kB39.0 kB+1.50 kB (+4.0%)
    storage (list + listAll)36.9 kB38.4 kB+1.50 kB (+4.1%)
    storage (updateMetadata)37.7 kB39.2 kB+1.50 kB (+4.0%)
    storage (uploadBytes)42.3 kB43.8 kB+1.50 kB (+3.6%)
    storage (uploadBytesResumable)51.7 kB53.2 kB+1.50 kB (+2.9%)
    storage (uploadString)42.5 kB44.0 kB+1.50 kB (+3.5%)

  • firebase

    15 size changes

    TypeBase (5c92d9c)Merge (acb4fe5)Diff
    firebase-analytics-compat.js24.3 kB25.6 kB+1.26 kB (+5.2%)
    firebase-analytics.js105 kB111 kB+5.89 kB (+5.6%)
    firebase-app-compat.js26.4 kB27.6 kB+1.26 kB (+4.8%)
    firebase-app.js81.5 kB87.4 kB+5.90 kB (+7.2%)
    firebase-auth-react-native.js492 kB487 kB-4.98 kB (-1.0%)
    firebase-compat.js780 kB781 kB+1.10 kB (+0.1%)
    firebase-messaging-compat.js36.4 kB37.5 kB+1.11 kB (+3.0%)
    firebase-messaging-sw.js101 kB107 kB+5.89 kB (+5.8%)
    firebase-messaging.js99.4 kB105 kB+5.89 kB (+5.9%)
    firebase-performance-compat.js29.1 kB30.4 kB+1.25 kB (+4.3%)
    firebase-performance-standalone-compat.es2017.js86.5 kB87.1 kB+626 B (+0.7%)
    firebase-performance-standalone-compat.js64.0 kB64.9 kB+925 B (+1.4%)
    firebase-performance.js117 kB123 kB+5.89 kB (+5.0%)
    firebase-remote-config-compat.js25.8 kB27.1 kB+1.26 kB (+4.9%)
    firebase-remote-config.js106 kB112 kB+5.89 kB (+5.5%)

Test Logs

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

@google-oss-bot
Copy link
Contributor

google-oss-bot commented May 3, 2022

Size Analysis Report 1

This report is too large (148,440 characters) to be displayed here in a GitHub comment. Please use the below link to see the full report on Google Cloud Storage.

Test Logs

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

Copy link
Member

@lahirumaramba lahirumaramba left a comment

Choose a reason for hiding this comment

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

LGTM! Left a few comments

contents: write
steps:
- name: Merge to master
uses: actions/[email protected]
Copy link
Member

Choose a reason for hiding this comment

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

Looks like the latest version is v6 now (includes breaking changes). Worth checking if we can use one of the recent versions here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will try to bump to v6

runs-on: ubuntu-latest
# Allow GITHUB_TOKEN to have write permissions
permissions:
contents: write
Copy link
Member

Choose a reason for hiding this comment

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

Is the goal here to grant GITHUB_TOKEN with default permissions (without write access) and add write access in the workflow?

@@ -0,0 +1,22 @@
name: Merge Release Into Master

on: repository_dispatch
Copy link
Member

Choose a reason for hiding this comment

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

nit feel free to ignore:
We can set types: here if we want to restrict the manual trigger even further with a specific event_type value in the webhook payload.

on:
  repository_dispatch:
    types: [sdk-release-merge]

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will add this when/if we trigger from a webhook. We'll use the button in the Github UI for now.

@hsubox76 hsubox76 merged commit 5ce0676 into master May 4, 2022
@hsubox76 hsubox76 deleted the ch-release-merge branch May 4, 2022 20:07
@firebase firebase locked and limited conversation to collaborators Jun 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants