Skip to content

Update changelog#1858

Merged
wandamora merged 1 commit intomasterfrom
wm-update-changelog
Apr 3, 2026
Merged

Update changelog#1858
wandamora merged 1 commit intomasterfrom
wm-update-changelog

Conversation

@wandamora
Copy link
Copy Markdown
Contributor

Changelog was missing recent updates.

@wandamora wandamora requested a review from shettyvarun268 April 3, 2026 18:19
Copy link
Copy Markdown
Contributor

@shettyvarun268 shettyvarun268 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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 recent changes. The review suggests restructuring the file with a top-level heading and version section, as well as improving the formatting and capitalization of the entries for better consistency.

Comment thread CHANGELOG.md
Comment on lines +1 to +2
- Accept BooleanParams in CallableOptions (#1854)
- rotate the npmrc key to robot account (#1857)
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 CHANGELOG.md file lacks a standard structure. It should include a top-level heading and a version section (e.g., ## Unreleased) to organize the entries. Additionally, the second entry should be capitalized for consistency and use .npmrc to correctly refer to the configuration file.

Suggested change
- Accept BooleanParams in CallableOptions (#1854)
- rotate the npmrc key to robot account (#1857)
# Changelog
## Unreleased
- Accept BooleanParams in CallableOptions (#1854)
- Rotate the .npmrc key to robot account (#1857)

@wandamora wandamora marked this pull request as ready for review April 3, 2026 18:22
@wandamora wandamora added this pull request to the merge queue Apr 3, 2026
Merged via the queue into master with commit 4d6a1cf Apr 3, 2026
32 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.

3 participants