Skip to content
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1 +1,3 @@
- Updated the Firebase Data Connect local toolkit to v3.3.4, which includes the following changes:
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.

high

The pull request title and this changelog entry indicate an update to the Firebase Data Connect local toolkit to v3.3.4, but the actual code changes (e.g., version bump in package.json or a version constant file) are missing from this PR.

Additionally, the PR description is empty, which violates the repository style guide requiring ### Description, ### Scenarios Tested, and ### Sample Commands.

References
  1. The repository style guide requires PR descriptions to include specific sections like Description, Scenarios Tested, and Sample Commands to provide context and verification of changes. (link)

- [fixed] Fixed an issue where Angular SDK generation would fail.
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.

medium

The phrasing [fixed] Fixed is redundant and inconsistent with the existing changelog entries. Furthermore, if this update introduces a new angularSdk target, please verify if src/commands/dataconnect-sdk-generate.ts needs to be updated to include it in the loadAllWithSDKs filter, as it currently only checks for other SDK types.

Suggested change
- [fixed] Fixed an issue where Angular SDK generation would fail.
- Fixed an issue where Angular SDK generation would fail.

- Fixed an issue where functions deployments would silently fail (#6989)
Loading