-
-
Notifications
You must be signed in to change notification settings - Fork 59
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
Error with tslib
when trying to firebase deploy
#382
Comments
Also going through testing the fix in a full project which deploys through CI as well |
Cool I'm actually trying that now too, I haven't used TravisCI before so trying to figure out how to get it to trigger a build. Let me know if you stumble upon any gotchas or things that you do that aren't documented please! |
actually I'm not sure it's even setup right to run with Travis, seems the builds are all self-contained in Github actions (another thing I haven't used yet). some more documentation on how to setup CI would be helpful IMHO. regardless, my github deploy is action is failing with: And I found that I needed to set |
If you haven't used a CI or have a strong preference I would highly suggest using Github Actions Yeah seeing that myself as well - there was a change to Github Action's syntax for setting env variables working on addressing that now |
Forgot to post, but released 7.6.6 with the fix for both the tslib and environment variable form: https://github.com/prescottprue/generator-react-firebase/releases/tag/v7.6.6 |
this is working for you with github actions? I re-installed the generator, started from scratch with a new project, created the firebase token with |
@prescottprue I opened a PR at #384 to fix the env issue I'm seeing. |
Didn't get a chance to test the full actions flow yesterday Nope, you aren't doing anything wrong, I just never came back to fully update that Syntax after it changed on the Github Actions side. I'm going to do a bit more testing here to make sure the CI flows are working. Thanks again, super helpful! |
@prescottprue Any updates on the testing? With these changes the github action now succeeds for me but there is something wrong with the deploy itself as I end up with a blank page and |
Hey @mikeover I forgot to update, but i went through with the testing and didn't have an issue in a test project, here is the successful deploy. What is the branch name for the build that was run? Do you have Did you check the CI logs to see which configs were loaded? This is dependent on matching aliases in |
Describe the bug
I was not able to do
firebase deploy
because an error with the functions regardingtslib
. I added"tslib": "2.2.0"
to the dependencies infunctions/package.json
and then was able to deploy. Separate issue thoughTo Reproduce
Same steps as from #369. I went through the manual deploy steps and once I got to
firebase deploy
it failed on deploying the functions until I addedtslib
to the package.json.The text was updated successfully, but these errors were encountered: