Skip to content

chore: revert update .nycrc, Contributing.md, and .gitignore across packages - #9354

Merged
quirogas merged 1 commit into
mainfrom
revert-9144-pnpm-config-updates
Sep 16, 2026
Merged

quirogas merged 1 commit into
mainfrom
revert-9144-pnpm-config-updates

Conversation

@bshaffer

@bshaffer bshaffer commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Partially revert #9144 by reverting all changes to the packages file. This changes will have to be propagated through the generator by releasing a new version of the generator, updating librarian.yaml, and re-generating all packages.

@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 transitions the workspace from pnpm to npm, updating installation and testing instructions in CONTRIBUTING.md files, adding package-lock.json to .gitignore files, and updating .nycrc configurations to include the 'text' reporter. Feedback on the changes highlights an asynchronous fs.unlink call in a synchronous afterEach hook in log-sync.ts that could lead to test flakiness, as well as a duplicate package-lock.json entry in the .gitignore file for gcp-metadata.

Comment thread handwritten/logging/test/log-sync.ts
Comment thread core/packages/gcp-metadata/.gitignore Outdated
@bshaffer
bshaffer force-pushed the revert-9144-pnpm-config-updates branch from 8c91f35 to fb90b48 Compare September 16, 2026 16:10
@quirogas
quirogas marked this pull request as ready for review September 16, 2026 16:59
@quirogas
quirogas requested a review from a team as a code owner September 16, 2026 16:59
@github-actions
github-actions Bot requested a review from feywind September 16, 2026 16:59
@quirogas
quirogas merged commit 65a9440 into main Sep 16, 2026
87 checks passed
@quirogas
quirogas deleted the revert-9144-pnpm-config-updates branch September 16, 2026 17:02
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