Skip to content
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
4 changes: 3 additions & 1 deletion .nycrc
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": [
"lcov"
],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
6 changes: 3 additions & 3 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -342,14 +342,14 @@ with an issue in this repository.

1. Install dependencies:

npm install
pnpm install

1. Run the tests:

npm test
pnpm run test

1. Lint (and maybe fix) any changes:

npm run fix
pnpm run fix

[setup]: https://cloud.google.com/nodejs/docs/setup
1 change: 0 additions & 1 deletion core/common/.gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -10,5 +10,4 @@ system-test/*key.json
*-lock.js*
build/
.vscode
package-lock.json
__pycache__
4 changes: 3 additions & 1 deletion core/common/.nycrc
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": [
"lcov"
],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
6 changes: 3 additions & 3 deletions core/common/CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -50,12 +50,12 @@ accept your pull requests.

1. Install dependencies:

npm install
pnpm install

1. Run the tests:

# Run unit tests.
npm test
pnpm run test

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

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.


# Run sample integration tests.
npm run samples-test
Expand All @@ -65,7 +65,7 @@ accept your pull requests.

1. Lint (and maybe fix) any changes:

npm run fix
pnpm run fix

[setup]: https://cloud.google.com/nodejs/docs/setup
[projects]: https://console.cloud.google.com/project
Expand Down
1 change: 0 additions & 1 deletion core/dev-packages/jsdoc-fresh/.gitignore
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
node_modules
package-lock.json
__pycache__
docs
.DS_Store
4 changes: 3 additions & 1 deletion core/dev-packages/jsdoc-fresh/.nycrc
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": [
"lcov"
],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
6 changes: 3 additions & 3 deletions core/dev-packages/jsdoc-fresh/CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -50,12 +50,12 @@ accept your pull requests.

1. Install dependencies:

npm install
pnpm install

1. Run the tests:

# Run unit tests.
npm test
pnpm run test

# Run sample integration tests.
npm run samples-test
Expand All @@ -65,7 +65,7 @@ accept your pull requests.

1. Lint (and maybe fix) any changes:

npm run fix
pnpm run fix

[setup]: https://cloud.google.com/nodejs/docs/setup
[projects]: https://console.cloud.google.com/project
Expand Down
1 change: 0 additions & 1 deletion core/dev-packages/jsdoc-region-tag/.gitignore
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
node_modules
.DS_Store
coverage
package-lock.json
.coverage
docs
4 changes: 3 additions & 1 deletion core/dev-packages/jsdoc-region-tag/.nycrc
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": [
"lcov"
],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
6 changes: 3 additions & 3 deletions core/dev-packages/jsdoc-region-tag/CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -50,12 +50,12 @@ accept your pull requests.

1. Install dependencies:

npm install
pnpm install

1. Run the tests:

# Run unit tests.
npm test
pnpm run test

# Run sample integration tests.
npm run samples-test
Expand All @@ -65,7 +65,7 @@ accept your pull requests.

1. Lint (and maybe fix) any changes:

npm run fix
pnpm run fix

[setup]: https://cloud.google.com/nodejs/docs/setup
[projects]: https://console.cloud.google.com/project
Expand Down
2 changes: 0 additions & 2 deletions core/dev-packages/pack-n-play/.gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -2,5 +2,3 @@ build
node_modules
.nyc_output
.coverage/*
test/fixtures/**/package-lock.json
package-lock.json
4 changes: 3 additions & 1 deletion core/dev-packages/pack-n-play/.nycrc
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": [
"lcov"
],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
6 changes: 3 additions & 3 deletions core/dev-packages/pack-n-play/CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -50,12 +50,12 @@ accept your pull requests.

1. Install dependencies:

npm install
pnpm install

1. Run the tests:

# Run unit tests.
npm test
pnpm run test

# Run sample integration tests.
npm run samples-test
Expand All @@ -65,7 +65,7 @@ accept your pull requests.

1. Lint (and maybe fix) any changes:

npm run fix
pnpm run fix

[setup]: https://cloud.google.com/nodejs/docs/setup
[projects]: https://console.cloud.google.com/project
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -52,12 +52,12 @@ accept your pull requests.

1. Install dependencies:

npm install
pnpm install

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

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
  1. 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.


1. Run the tests:

# Run unit tests.
npm test
pnpm run test

# Run sample integration tests.
npm run samples-test
Expand All @@ -67,7 +67,7 @@ accept your pull requests.

1. Lint (and maybe fix) any changes:

npm run fix
pnpm run fix

[setup]: https://cloud.google.com/nodejs/docs/setup
[projects]: https://console.cloud.google.com/project
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"report-dir": "./.coverage",
"reporter": ["text", "lcov"],
"reporter": ["lcov"],
"exclude": [
"**/*-test",
"**/.coverage",
Expand Down
Loading
Loading