-
Notifications
You must be signed in to change notification settings - Fork 424
chore(storage-resize-images): migration to Node.js v14 #657
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
chore(storage-resize-images): migration to Node.js v14 #657
Conversation
Codecov Report
@@ Coverage Diff @@
## next #657 +/- ##
=======================================
Coverage 50.65% 50.65%
=======================================
Files 30 30
Lines 1313 1313
Branches 193 193
=======================================
Hits 665 665
Misses 647 647
Partials 1 1 |
storage-resize-images/README.md
Outdated
| - This extension uses other Firebase and Google Cloud Platform services, which have associated charges if you exceed the service’s free tier: | ||
| - Cloud Storage | ||
| - Cloud Functions (Node.js 10+ runtime. [See FAQs](https://firebase.google.com/support/faq#expandable-24)) | ||
| - Cloud Functions (Node.js 12+ runtime. [See FAQs](https://firebase.google.com/support/faq#expandable-24)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was curious so I checked. New URL: https://firebase.google.com/support/faq#extensions-pricing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great spot Mike, reading that article suggest that this change probably isn't needed as part of the PR? As the document specifically notes v10+..?
Could always update that link as a separate PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mmm good point @dackers86 - probably leave 10+ runtime in the comment (since that is the version in the pricing document) but definitely point to the new URL
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated.
Co-authored-by: Jeff <jhuleatt@google.com>
* chore(delete-user-data): refactor test data into delete user data (#655) * feat(*): added warsaw (europe-central2) location (#677) Co-authored-by: Rosário P. Fernandes <rosariofernandes51@gmail.com> * chore(storage-resize-images): migration to Node.js v14 (#657) * chore(storage-resize-images): migration to Node.js v12 * Undo readme, update preinstall * Documentation and package updates * Update storage-resize-images/README.md Co-authored-by: Jeff <jhuleatt@google.com> * node 12 to 14 Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com> Co-authored-by: Jeff <jhuleatt@google.com> * chore(delete-user-data): migration to Node.js v14 (#661) * chore(delete-user-data): migration to Node.js v12 * Updated engine to node14, build fixes and updated target lib * chore(firestore-counter): migration to Node.js v14 (#664) * chore(firestore-counter): migration to Node.js v12 * Updated engine to node14, build fixes and updated target lib * chore(firestore-send-email): migration to Node.js v14 (#665) * chore(firestore-send-email): migration to Node.js v12 * Updated engine to node14, build fixes and updated target lib * chore(firestore-shorten-urls-bitly): migration to Node.js v14 (#667) * chore(firestore-send-email): migration to Node.js v12 * chore(firestore-shortern-urls-bitly): migration to Node.js v12 * Updated engine to node14, build fixes and updated target lib * reverted send email updates * reverted additional send email updates * chore(firestore-translate-text): migration to Node.js v14 (#668) * chore(firestore-translate-text): migration to Node.js v12 * Updated engine to node14, build fixes and updated target lib * chore(rtdb-limit-child-nodes): migration to Node.js v14 (#669) * chore(rtdb-limit-child-nodes-node12): migration to Node.js v12 * Updated engine to run node14 * Updated target mapping and build fixes * chore(firestore-bigquery-export): migration to Node.js v14 (#662) * chore(firestore-bigquery-export): migration to Node.js v12 * Reverted change tracker updates, updated export dependencies * Updated engine to node14, build fixes and updated target lib * Update firestore-bigquery-export/functions/tsconfig.json Co-authored-by: Jeff <jhuleatt@google.com> Co-authored-by: Pavel Jbanov <pavelgj@gmail.com> * chore(*): regenerate release notes for June 21 2021 release (#680) Co-authored-by: Darren Ackers <ackers86@hotmail.com> Co-authored-by: Rosário P. Fernandes <rosariofernandes51@gmail.com> Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com> Co-authored-by: Jeff <jhuleatt@google.com>
Checklist
Testing
Jest Tests
N/a for this module.
Updated engine
Successful Installation
Successful result following installation