Skip to content

Upgrade typescript version to v6 in templates - #10232

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

joehan merged 4 commits into
firebase:mainfrom
wandamora:wm-update-template-ts-version

Conversation

@wandamora

@wandamora wandamora commented Mar 31, 2026 •

Copy link
Copy Markdown
Contributor

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

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

Copy link
Copy Markdown
Contributor

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

joehan commented Apr 2, 2026

Copy link
Copy Markdown
Member

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