chore: update .nycrc, Contributing.md, and .gitignore across packages - #9144
chore: update .nycrc, Contributing.md, and .gitignore across packages#9144bshaffer wants to merge 108 commits into
Conversation
…ng minimatch in c8
…pendencies are installed
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
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 |
There was a problem hiding this comment.
In this file (and many other CONTRIBUTING.md files across the repository), the command npm run samples-test on line 61 was missed during the migration from npm to pnpm. Please update it to pnpm run samples-test for consistency.
| 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.
…haustion and restore generator-tests
# Please enter a commit message to explain why this merge is necessary, # especially if it merges an updated upstream into a topic branch. # # Lines starting with '#' will be ignored, and an empty message aborts # the commit.
afc9e58 to
5b45a3c
Compare
… test/linter failures
…npm overrides across packages
c1db173 to
269e759
Compare
5ba1875 to
e2dbfe6
Compare
1 participant
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