chore: update .nycrc, Contributing.md, and .gitignore across packages - #9144
chore: update .nycrc, Contributing.md, and .gitignore across packages#9144bshaffer wants to merge 2 commits into
Conversation
There was a problem hiding this comment.
Code Review
This pull request migrates package management and scripts from npm to pnpm across multiple packages, updating CONTRIBUTING.md files, removing package-lock.json and yarn.lock from .gitignore files, and simplifying coverage reporters in .nycrc files. Feedback highlights that some instances of npm run samples-test were missed in the CONTRIBUTING.md updates, and warns against manually editing auto-generated baseline and test-fixture files instead of updating the upstream generator templates.
|
|
||
| # Run unit tests. | ||
| npm test | ||
| pnpm run test |
| 1. Install dependencies: | ||
|
|
||
| npm install | ||
| pnpm install |
There was a problem hiding this comment.
It appears that CONTRIBUTING.md (and potentially .gitignore files) in the baseline and test-fixture directories are being manually updated here. According to the repository rules, manual edits to auto-generated files or baseline files should be avoided as they will be overwritten during the next regeneration. Please ensure that the upstream generator templates (or the source files used by the generator) are updated instead, and then regenerate these baseline/test-fixture files.
References
- Do not manually edit auto-generated files to fix typos or make other changes, as these edits will be overwritten during the next regeneration. Instead, apply the fixes upstream in the generator or templates.
…npm overrides across packages
afc9e58 to
5b45a3c
Compare
…ufjs v8 in gapic-generator-typescript
Separated from #8973 to keep the core PNPM migration PR focused.
This PR applies the changes across subpackages for:
.nycrcconfiguration updatesCONTRIBUTING.mdinstructions (updating npm -> pnpm).gitignorecleanups across packages