Skip to content

doc(dart_firebase_admin): s/undefined/null#211

Merged
demolaf merged 2 commits intofirebase:mainfrom
brianquinlan:no_undefined
Apr 10, 2026
Merged

doc(dart_firebase_admin): s/undefined/null#211
demolaf merged 2 commits intofirebase:mainfrom
brianquinlan:no_undefined

Conversation

@brianquinlan
Copy link
Copy Markdown
Collaborator

Removes the use of undefined in dartdoc (undefined is not a type in Dart).

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 updates documentation across several files to replace JavaScript-specific terminology like 'undefined' with Dart-appropriate terms such as 'null' or 'not specified'. Additionally, it refactors a code example in document.dart to use async/await syntax. A review comment points out that the updated code example incorrectly uses JSON.stringify, which is not standard Dart, and suggests using jsonEncode from dart:convert instead.

@codecov-commenter
Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@Ehesp
Copy link
Copy Markdown
Member

Ehesp commented Apr 9, 2026

Thats the javascripter coming through!

@demolaf demolaf merged commit 560a379 into firebase:main Apr 10, 2026
10 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.

4 participants