Skip to content

chore: remove temporary Dart console warning#10584

Merged
demolaf merged 3 commits into
mainfrom
dart-console-support
Jun 11, 2026
Merged

chore: remove temporary Dart console warning#10584
demolaf merged 3 commits into
mainfrom
dart-console-support

Conversation

@demolaf

@demolaf demolaf commented Jun 2, 2026

Copy link
Copy Markdown
Member

The Firebase Console now supports Dart functions, so the temporary warning directing users to the Cloud Console is no longer needed.

@demolaf demolaf requested a review from ajperel June 2, 2026 09:43

@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 removes the temporary Dart runtime check and its associated warning message during the functions release process. The reviewer correctly noted that removing this check leaves the import of runtimeIsLanguage unused, which should be cleaned up.

I am having trouble creating individual review comments. Click here to see my feedback.

src/deploy/functions/release/index.ts (118-127)

medium

Since the Dart runtime check has been removed, the import of runtimeIsLanguage on line 20 is now unused and should be removed.

@demolaf demolaf force-pushed the dart-console-support branch from 6ee05ca to ac84cf0 Compare June 3, 2026 08:30

@shettyvarun268 shettyvarun268 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.

LGTM.
You might want to add an entry in the change log but overall looks okay!

@demolaf demolaf force-pushed the dart-console-support branch 3 times, most recently from b4342af to c9b52c8 Compare June 11, 2026 18:06
@demolaf demolaf enabled auto-merge (squash) June 11, 2026 18:07
@demolaf demolaf force-pushed the dart-console-support branch from c9b52c8 to 8984c0c Compare June 11, 2026 20:03
@demolaf demolaf merged commit 88339cf into main Jun 11, 2026
51 checks passed
@demolaf demolaf deleted the dart-console-support branch June 11, 2026 20:16
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.

4 participants