Skip to content

Update CHANGELOG.md for #10503#10513

Closed
Berlioz wants to merge 1 commit into
mainfrom
Berlioz-patch-1
Closed

Update CHANGELOG.md for #10503#10513
Berlioz wants to merge 1 commit into
mainfrom
Berlioz-patch-1

Conversation

@Berlioz
Copy link
Copy Markdown
Contributor

@Berlioz Berlioz commented May 13, 2026

Forgot to update it in the PR itself.

Forgot to update it in the PR itself.
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 the CHANGELOG.md to include a new entry regarding the automatic upserting of IAM bindings for AI Logic functions during deployment. The review feedback suggests a minor stylistic refinement to the changelog entry to improve professionalism and consistency by removing redundant language.

Comment thread CHANGELOG.md
- Added support for nested 1:Many relational batch mutations.
- Updated the Golang dependency version to 1.25.10.
- Default timeout for Dart functions is now 60 seconds when not explicitly set (#10501)
- Now upserts the required IAM bindings to invoke AI Logic functions upon deployment (#10503)
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 word "Now" is redundant in a changelog entry. It is more professional and consistent with other entries in this file to start directly with a verb.

Suggested change
- Now upserts the required IAM bindings to invoke AI Logic functions upon deployment (#10503)
- Automatically upserts the required IAM bindings to invoke AI Logic functions upon deployment (#10503)

@Berlioz Berlioz requested review from ajperel and shettyvarun268 May 13, 2026 20:05
@Berlioz
Copy link
Copy Markdown
Contributor Author

Berlioz commented May 13, 2026

Redundant with #10514

@Berlioz Berlioz closed this May 13, 2026
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.

2 participants