Skip to content

Commit 279ad98

Browse files
author
Michael Brewer
authored
fix(build): Update contributing.md and fix npm ci (#417)
* fix(build): Update contributing.md and fix npm ci Changes: - Update CONTRIBUTING.md to use > [email protected] preinstall > (cd packages/commons && npm ci); (cd packages/logger && npm ci); (cd packages/metrics && npm ci); (cd packages/tracing && npm ci); > @aws-lambda-powertools/[email protected] prepare > npm run build > @aws-lambda-powertools/[email protected] build > tsc added 586 packages, and audited 587 packages in 4s 84 packages are looking for funding run `npm fund` for details found 0 vulnerabilities > @aws-lambda-powertools/[email protected] prepare > npm run build > @aws-lambda-powertools/[email protected] build > tsc added 488 packages, and audited 489 packages in 4s 72 packages are looking for funding run `npm fund` for details found 0 vulnerabilities > @aws-lambda-powertools/[email protected] prepare > npm run build > @aws-lambda-powertools/[email protected] build > tsc added 822 packages, and audited 852 packages in 10s 91 packages are looking for funding run `npm fund` for details found 0 vulnerabilities > @aws-lambda-powertools/[email protected] prepare > npm run build > @aws-lambda-powertools/[email protected] build > tsc added 836 packages, and audited 859 packages in 8s 72 packages are looking for funding run `npm fund` for details found 0 vulnerabilities added 1327 packages, removed 2761 packages, changed 10 packages, and audited 1383 packages in 40s 100 packages are looking for funding run `npm fund` for details found 0 vulnerabilities - Add a preinstall task to install all of the sub packages closes #415 * fix(build): Update contributing.md and fix npm ci Changes: - Update CONTRIBUTING.md to use npm ci - Add a preinstall task to install all of the sub packages closes #415 * chore: leave as npm install for now * fix(all): setup up project references * docs: update to use npm ci
1 parent fd02acb commit 279ad98

File tree

5 files changed

+22
-4
lines changed

5 files changed

+22
-4
lines changed

Diff for: CONTRIBUTING.md

+6-4
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ Please read through this document before submitting any issues or pull requests
77
information to effectively respond to your bug report or contribution.
88

99
## Security issue notifications
10+
1011
If you discover a potential security issue in this project we ask that you notify AWS/Amazon Security via our [vulnerability reporting page](http://aws.amazon.com/security/vulnerability-reporting/). Please do **not** create a public GitHub issue.
1112

1213
## Reporting Bugs/Feature Requests
@@ -35,7 +36,7 @@ Contributions via pull requests are much appreciated. Before sending us a pull r
3536
To send us a pull request, please follow these steps:
3637

3738
1. Fork the repository.
38-
2. Install dependencies: `npm install`
39+
2. Install dependencies: `npm ci`
3940
3. Prepare utilities like commit hooks: `npm run init-environment`
4041
4. Create a new branch to focus on the specific change you are contributing e.g. `git checkout -b improv/logger-debug-sampling`
4142
5. Run all tests, and code baseline checks: `npm run test`
@@ -54,8 +55,9 @@ You might find useful to run both the documentation website and the API referenc
5455
* **Docs website**:
5556

5657
You can build and start a local docs website by running these two commands.
57-
- `npm run docs-buildDockerImage` OR `docker build -t squidfunk/mkdocs-material ./docs/`
58-
- `npm run docs-runLocalDocker` OR `docker run --rm -it -p 8000:8000 -v ${PWD}:/docs squidfunk/mkdocs-material`
58+
59+
* `npm run docs-buildDockerImage` OR `docker build -t squidfunk/mkdocs-material ./docs/`
60+
* `npm run docs-runLocalDocker` OR `docker run --rm -it -p 8000:8000 -v ${PWD}:/docs squidfunk/mkdocs-material`
5961

6062
### Conventions
6163

@@ -89,4 +91,4 @@ TODO
8991

9092
See the [LICENSE](LICENSE) file for our project's licensing. We will ask you to confirm the licensing of your contribution.
9193

92-
We may ask you to sign a [Contributor License Agreement (CLA)](http://en.wikipedia.org/wiki/Contributor_License_Agreement) for larger changes.
94+
We may ask you to sign a [Contributor License Agreement (CLA)](http://en.wikipedia.org/wiki/Contributor_License_Agreement) for larger changes.

Diff for: packages/commons/tsconfig.json

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
{
22
"compilerOptions": {
3+
"composite": true,
34
"experimentalDecorators": true,
45
"noImplicitAny": true,
56
"target": "ES2020",

Diff for: packages/logger/tsconfig.json

+5
Original file line numberDiff line numberDiff line change
@@ -26,5 +26,10 @@
2626
"types": [
2727
"jest",
2828
"node"
29+
],
30+
"references": [
31+
{
32+
"path": "../commons"
33+
}
2934
]
3035
}

Diff for: packages/metrics/tsconfig.json

+5
Original file line numberDiff line numberDiff line change
@@ -28,5 +28,10 @@
2828
"types": [
2929
"jest",
3030
"node"
31+
],
32+
"references": [
33+
{
34+
"path": "../commons"
35+
}
3136
]
3237
}

Diff for: packages/tracing/tsconfig.json

+5
Original file line numberDiff line numberDiff line change
@@ -28,5 +28,10 @@
2828
"types": [
2929
"jest",
3030
"node"
31+
],
32+
"references": [
33+
{
34+
"path": "../commons"
35+
}
3136
]
3237
}

0 commit comments

Comments
 (0)