Skip to content
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

Add repository fields to some compat packages and add workflow trigger for canary #5587

Merged
merged 3 commits into from
Oct 11, 2021

Conversation

hsubox76
Copy link
Contributor

@hsubox76 hsubox76 commented Oct 8, 2021

database-compat, firestore-compat, and storage-compat were missing repository fields, which wombat-dressing-room uses to determine if we have permission to publish. This breaks canary and prerelease workflows, which use wombat-dressing room to publish packages.

I've also added a workflow_dispatch trigger to the canary workflow to make it easier to manually trigger a canary build in the future for debugging or other reasons.

@changeset-bot
Copy link

changeset-bot bot commented Oct 8, 2021

🦋 Changeset detected

Latest commit: 86e999c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages
Name Type
@firebase/database-compat Patch
@firebase/firestore-compat Patch
@firebase/storage-compat Patch
firebase Patch

Not sure what this means? Click here to learn what changesets are.

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

@google-cla google-cla bot added the cla: yes label Oct 8, 2021
@google-oss-bot
Copy link
Contributor

google-oss-bot commented Oct 8, 2021

Binary Size Report

Affected SDKs

No changes between base commit (53a9263) and head commit (69539eb).

Test Logs

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Oct 8, 2021

Size Analysis Report

Affected Products

No changes between base commit (53a9263) and head commit (69539eb).

@@ -4,6 +4,7 @@ on:
push:
branches:
- master
workflow_dispatch:
Copy link
Member

Choose a reason for hiding this comment

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

Is it okay to have a colon with an empty body?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Feiyang1 Feiyang1 assigned hsubox76 and unassigned Feiyang1 Oct 11, 2021
@hsubox76 hsubox76 merged commit 352cc26 into master Oct 11, 2021
@hsubox76 hsubox76 deleted the ch-canary-trigger branch October 11, 2021 17:29
@google-oss-bot google-oss-bot mentioned this pull request Oct 12, 2021
@firebase firebase locked and limited conversation to collaborators Nov 11, 2021
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