Skip to content

Commit 13182e0

Browse files
committed
Merge branch 'feature/split' into develop
2 parents 2d3e49d + dc0ce3b commit 13182e0

File tree

228 files changed

+11478
-7175
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

228 files changed

+11478
-7175
lines changed

.babelrc

Lines changed: 0 additions & 15 deletions
This file was deleted.

.eslintignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
11
node_modules
22
coverage
33
dist
4+
jest.config.js

.eslintrc

Lines changed: 0 additions & 32 deletions
This file was deleted.

.github/workflows/monorepo.yml

Lines changed: 53 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,53 @@
1+
on:
2+
push:
3+
branches:
4+
- feature/split
5+
6+
name: monorepo
7+
8+
jobs:
9+
do-de-stuff:
10+
runs-on: ubuntu-latest
11+
steps:
12+
- uses: actions/checkout@v2
13+
- uses: actions/setup-node@v2
14+
- run: yarn
15+
- run: ./node_modules/.bin/lerna run lint
16+
- run: ./node_modules/.bin/lerna run test -- --coverage
17+
- name: Coveralls (signpdf)
18+
uses: coverallsapp/github-action@master
19+
with:
20+
parallel: true
21+
flag-name: signpdf
22+
github-token: ${{ secrets.GITHUB_TOKEN }}
23+
base-path: packages/signpdf
24+
path-to-lcov: packages/signpdf/coverage/lcov.info
25+
- name: Coveralls (utils)
26+
uses: coverallsapp/github-action@master
27+
with:
28+
parallel: true
29+
flag-name: utils
30+
github-token: ${{ secrets.GITHUB_TOKEN }}
31+
base-path: packages/utils
32+
path-to-lcov: packages/utils/coverage/lcov.info
33+
- name: Coveralls (placeholder-pdfkit010)
34+
uses: coverallsapp/github-action@master
35+
with:
36+
parallel: true
37+
flag-name: placeholder-pdfkit010
38+
github-token: ${{ secrets.GITHUB_TOKEN }}
39+
base-path: packages/placeholder-pdfkit010
40+
path-to-lcov: packages/placeholder-pdfkit010/coverage/lcov.info
41+
- name: Coveralls (placeholder-plain)
42+
uses: coverallsapp/github-action@master
43+
with:
44+
parallel: true
45+
flag-name: placeholder-plain
46+
github-token: ${{ secrets.GITHUB_TOKEN }}
47+
base-path: packages/placeholder-plain
48+
path-to-lcov: packages/placeholder-plain/coverage/lcov.info
49+
- name: Close Coveralls
50+
uses: coverallsapp/github-action@master
51+
with:
52+
parallel-finished: true
53+
carryforward: "signpdf,utils,placeholder-pdfkit010,placeholder-plain"

.github/workflows/not-travis.yml

Lines changed: 0 additions & 15 deletions
This file was deleted.

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ yarn-error.log*
99

1010
coverage
1111
.coveralls.yml
12+
packages/examples/output
1213

1314
node_modules/
1415
.npm

CHANGELOG.md

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,15 @@
11
# CHANGELOG
22

3+
## [3.0.0]
4+
5+
* Deprecating `node-signpdf` to be replaced by the `@signpdf` scoped packages;
6+
* Split the helpers into separate packages `@signpdf/placeholder-pdfkit010` and `@signpdf/placeholder-plain`;
7+
* Decoupled the signing algorythm and the PDF signing process. Now `@signpdf/signpdf` understands PDFs and `@signpdf/signer-p12` understands P12 and `node-forge`;
8+
* sign() is now async to allow for hooking different signers;
9+
* placeholder helpers now require `contactInfo`, `location` and `name` which used to be optional;
10+
* Introduced examples in `packages/examples`;
11+
* Introduced more types declarations through TS and JSDoc;
12+
313
## [2.0.0]
414

515
* Only publish dist files into npm (see https://bit.ly/46rykd7 on why this requires a major version);

CONTRIBUTING.md

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,21 @@
11
# Contributing
22

3-
* All PRs are welcome in the `develop` branch.
4-
* Please, stick to the repo's defined [Purpose](README.md#purpose) and try to write your code as readable as possible.
3+
* **All PRs are welcome** in the `develop` branch.
4+
* Please, stick to the defined [Purpose](https://github.com/vbuch/node-signpdf#purpose) of the repo and try to write your code as readable as possible.
5+
* [Yarn Workspaces](https://classic.yarnpkg.com/lang/en/docs/workspaces/) + [Lerna](https://lerna.js.org/) are used in this **monorepo**.
56
* We do code reviews and may ask you to change things before we merge code.
6-
* This is a git-flow repo. We use the default git flow with a `v` version prefix. You shouldn't care much while writeing a feature but good to know.
77
* Note that [gitmoji](https://gitmoji.carloscuesta.me/) is used in the commit messages. That's not a must but we think it's nice.
88

9+
## Useful commands
10+
11+
- `$ yarn lint` - As run in your pre-commit
12+
- `$ yarn test` - As run in your pre-push
13+
- `$ ./node_modules/.bin/lerna run build` builds all the packages in their respective `dist` folders.
14+
- `$ ./node_modules/.bin/lerna run test --scope=@signpdf/signpdf -- --watch --coverage` runs tests in a specified package tracking coverage and watching.
15+
- `$ yarn workspace @signpdf/signpdf add node-forge -D` adds a `node-forge` as a dev dependency
16+
of `@signpdf/signpdf`
17+
- `$ yarn workspace @signpdf/signpdf add @signpdf/some-new-subpackage@* -D` adds a `@signpdf/some-new-subpackage` as a dev dependency of `@signpdf/signpdf`. Note the `@*` version specifier.
18+
919
## Contributors
1020

1121
* [vbuch](https://github.com/vbuch)

0 commit comments

Comments
 (0)