-
Notifications
You must be signed in to change notification settings - Fork 201
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
What is the point? #221
Comments
Lately I was thinking the same thing, because every time I update fairebase-tools, this actions breaks and stops continuous deployment. https://github.com/w9jds/firebase-action/blob/master/entrypoint.sh I think the point of this actions is to absorb differences in parameter settings, but I would like to hear the Contributer's opinion. So I have stopped using this actions, while thanking them for their contribution so far. My new code is as follows It works comfortably.
Points:
|
Incase anyone comes along later, I used the This is a short version of how it looks in my Github actions:
This is in the package.json script:
Switching to this version also helped me avoid some strange IAM errors that I could not overcome which looked like this:
|
Thanks for sharing. I was working on this a bit recently and learned that using service account credentials is no longer recommended for github actions. |
Yep I agree, but I'm also using Firebase Admin SDK which unfortunately does not support Workload Identity Federation. So I have to resort to service account credentials for now. The docs for the google-github-actions/auth@v2 action have a warning about it:
Also FYI if you're using Terraform to stand up Firebase resources you might also get caught up in that issue and you have to use service account credentials. I spent a couple fun days figuring that out. There is a solution hopefully incoming shortly. Here is the long running issue for anyone interested: firebase/firebase-admin-node#1377. There is a solution hopefully incoming shortly. |
Thanks that's good to know! I didn't integrate github actions yet. |
I think a main point of this package is so that you don't have to include firebase-tools as a dev dependency. Right now I'm using Bun.sh to deploy my project. The current version of Bun.sh has some issues with the firebase-tools package and breaks deployment. This package lets me get around that. |
Yeah, ideally there is no reason to have It's not mandatory to use, but it's pretty common best practice not to include stuff like this inside of your repo. Obviously, you do what is best for your use case. This was created to just allow you to easily configure and setup a firebase deployment, testing, etc. system without having to build everything in your repo. |
Our use case for this is convenience and to avoid having a service account key for the admin-sdk service account in our GitHub secrets (we disable service account key creation via org policy anyway). You can auth with
|
I am having a hard time seeing any benefits that this action brings.
In the past, I've defined
firebase-tools
as a dev dependency, and installed it in github actions with the other dependencies. Then you can execute any command as usual, or via a package.json script entry like- run: pnpm run deploy:prd
.What does this action allow me to do easily that would be problematic with my current approach?
The text was updated successfully, but these errors were encountered: