Skip to content

github: Replace firebase-deployment action#26

Merged
juliaogris merged 2 commits intomasterfrom
firebase-deployment
Oct 6, 2022
Merged

github: Replace firebase-deployment action#26
juliaogris merged 2 commits intomasterfrom
firebase-deployment

Conversation

@juliaogris
Copy link
Member

Replace firebase-deployment action by hand-crafted make targets. This
has been motivated by an apparently broken firebase-tools release
(4 hours old) that cannot be down-graded when using the firebase GitHub
action and is a blocker. Aside: the foxygoat authors believe it is nice
to stay third-party GitHub action free where possible so that the bulk
of what's run on CI can also be run locally.

While at it upgrade the firebase CLI version to the second most recent
one - firebase-11.13.0 works, but the previous firebase-11.6.0 and the
even more recent firebase-11.14.0 do not it seems.

Upgrade firebase version in hope to unbreak CI. `make firebase-deploy`
to dev channel also failed locally (randomly, it didn't do that before).
With the update it doesn't so 🤞.
@juliaogris juliaogris force-pushed the firebase-deployment branch from 2fa0f95 to 9ca3dd2 Compare October 6, 2022 02:42
Replace firebase-deployment action by hand-crafted make targets. This has
been motivated by an apparently broken firebase-tools release (4 hours
old) that cannot be down-graded when using the firebase GitHub action
and is a blocker. Aside: the foxygoat authors believe it is nice to
stay third-party GitHub action free where possible so that the bulk of
what's run on CI can also be run locally.
@juliaogris juliaogris force-pushed the firebase-deployment branch from 9ca3dd2 to c9105d5 Compare October 6, 2022 02:58
Copy link
Member

@camh- camh- left a comment

Choose a reason for hiding this comment

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

🍏 very nice

@juliaogris juliaogris merged commit f1abb80 into master Oct 6, 2022
@juliaogris juliaogris deleted the firebase-deployment branch October 6, 2022 03:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants