Skip to content

Upgrade typescript version to v6 in templates#10232

Merged
joehan merged 4 commits intofirebase:mainfrom
wandamora:wm-update-template-ts-version
Apr 2, 2026
Merged

Upgrade typescript version to v6 in templates#10232
joehan merged 4 commits intofirebase:mainfrom
wandamora:wm-update-template-ts-version

Conversation

@wandamora
Copy link
Copy Markdown
Contributor

@wandamora wandamora commented Mar 31, 2026

Description

Upgrade typescript version to 6.0 following https://devblogs.microsoft.com/typescript/announcing-typescript-6-0/.

Scenarios Tested

First ran in firebase-tools root.

npm run build
npm link

Then, followed up with making a clean directory and executing the following commands in the functions dir after firebase init functions.

npm run build
firebase deploy --only functions

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request attempts to update the TypeScript dependency to version 6.0.0 across several template configuration files. However, TypeScript 6.0.0 has not been released yet, and using this version will cause dependency installation failures. The reviewer has correctly identified this issue and suggested reverting to the current stable version, 5.7.3. Additionally, the pull request lacks the required description sections and a CHANGELOG.md entry as specified in the repository style guide.

Comment thread templates/extensions/typescript/package.lint.json
Comment thread templates/extensions/typescript/package.nolint.json
Comment thread templates/init/functions/typescript/package.lint.json
Comment thread templates/init/functions/typescript/package.nolint.json
Comment thread templates/extensions/typescript/package.lint.json
Comment thread templates/extensions/typescript/package.lint.json
@wandamora wandamora marked this pull request as ready for review March 31, 2026 23:52
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@wandamora wandamora linked an issue Mar 31, 2026 that may be closed by this pull request
@wandamora wandamora requested a review from joehan April 1, 2026 00:01
@wandamora wandamora force-pushed the wm-update-template-ts-version branch 2 times, most recently from 106a0b3 to db5578f Compare April 2, 2026 15:50
@wandamora
Copy link
Copy Markdown
Contributor Author

Hi @joehan, friendly ping on this :) let me know if there is a better reviewer.

@joehan
Copy link
Copy Markdown
Member

joehan commented Apr 2, 2026

LGTM, just add a changelog entry

Keeping typescript code here up-to-date.
Keeping typescript version up-to-date. Includes defining the rootDir to src in tsconfig.json.
@wandamora wandamora force-pushed the wm-update-template-ts-version branch from b3d5615 to 4ea78f0 Compare April 2, 2026 21:47
@wandamora wandamora enabled auto-merge (rebase) April 2, 2026 21:48
@joehan joehan disabled auto-merge April 2, 2026 22:47
@joehan joehan merged commit 2836661 into firebase:main Apr 2, 2026
14 checks passed
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.

Update Typecript version to v5 in templates

3 participants