Skip to content

refactor(functions): rename "npm_package" to "package" for kits:install - #10966

Merged
wandamora merged 3 commits into
mainfrom
morawand-kits-package-rename
Aug 20, 2026
Merged

refactor(functions): rename "npm_package" to "package" for kits:install#10966
wandamora merged 3 commits into
mainfrom
morawand-kits-package-rename

Conversation

@wandamora

Copy link
Copy Markdown
Contributor

No description provided.

@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 renames the npm_package option to package in the functions:kits:install command and updates the corresponding test suite. The review feedback highlights that package is a reserved keyword in JavaScript/TypeScript, which limits direct destructuring. Additionally, the reviewer suggests capitalizing the error message for consistency and updating the associated test assertions.

Comment thread src/commands/functions-kits-install.ts
Comment thread src/commands/functions-kits-install.ts
Comment thread src/commands/functions-kits-install.spec.ts
@wandamora
wandamora enabled auto-merge (squash) August 20, 2026 22:30
@wandamora
wandamora merged commit 8efed47 into main Aug 20, 2026
52 of 53 checks passed
@wandamora
wandamora deleted the morawand-kits-package-rename branch August 20, 2026 22:38
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