Skip to content

Commit

Permalink
chore(*): added LICENSE to all bundles, streamlined contents of the b…
Browse files Browse the repository at this point in the history
…undles, added npm workspaces (#30)

* chore(*): added LICENSE to all bundles, streamlined contents of the bundles, added npm workspaces, remove package-lock.json files, update npm scripts to use npm install instead of npm ci, remove root package.json on ci scripts
  • Loading branch information
aricart authored Jul 31, 2024
1 parent a7953ad commit 432bcf2
Show file tree
Hide file tree
Showing 47 changed files with 1,417 additions and 2,490 deletions.
9 changes: 6 additions & 3 deletions .github/workflows/core.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,10 @@ jobs:
with:
fetch-depth: 1

- name: Remove NPM Workspace
run: |
rm -f package.json
- name: Use Deno Version ${{ matrix.deno-version }}
uses: denoland/setup-deno@v1
with:
Expand Down Expand Up @@ -58,13 +62,12 @@ jobs:
NGS_CI_USER: ${{ secrets.NGS_CI_USER }}
run: |
deno task test
- name: Build NPM core bundle
working-directory: core
env:
CI: true
run: |
npm ci
npm install
npm run prepack
npm link
Expand All @@ -73,7 +76,7 @@ jobs:
env:
CI: true
run: |
npm ci
npm install
npm run prepack
npm link @nats-io/nats-core
npm test
8 changes: 6 additions & 2 deletions .github/workflows/jetstream.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,10 @@ jobs:
with:
fetch-depth: 1

- name: Remove NPM Workspace
run: |
rm -f package.json
- name: Use Deno Version ${{ matrix.deno-version }}
uses: denoland/setup-deno@v1
with:
Expand Down Expand Up @@ -54,7 +58,7 @@ jobs:
env:
CI: true
run: |
npm ci
npm install
npm run prepack
npm link
Expand All @@ -63,7 +67,7 @@ jobs:
env:
CI: true
run: |
npm ci
npm install
npm run prepack
npm link @nats-io/jetstream
npm test
8 changes: 6 additions & 2 deletions .github/workflows/kv.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,10 @@ jobs:
with:
fetch-depth: 1

- name: Remove NPM Workspace
run: |
rm -f package.json
- name: Use Deno Version ${{ matrix.deno-version }}
uses: denoland/setup-deno@v1
with:
Expand Down Expand Up @@ -54,7 +58,7 @@ jobs:
env:
CI: true
run: |
npm ci
npm install
npm run prepack
npm link
Expand All @@ -63,7 +67,7 @@ jobs:
env:
CI: true
run: |
npm ci
npm install
npm run prepack
npm link @nats-io/kv
npm test
6 changes: 5 additions & 1 deletion .github/workflows/npm_build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,10 @@ jobs:
with:
fetch-depth: 1

- name: Remove NPM Workspace
run: |
rm -f package.json
- name: Use Deno Version ${{ matrix.deno-version }}
uses: denoland/setup-deno@v1
with:
Expand All @@ -45,6 +49,6 @@ jobs:
CI: true
NGS_CI_USER: ${{ secrets.NGS_CI_USER }}
run: |
npm ci
npm install
npm run prepack
8 changes: 6 additions & 2 deletions .github/workflows/obj.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,10 @@ jobs:
with:
fetch-depth: 1

- name: Remove NPM Workspace
run: |
rm -f package.json
- name: Use Deno Version ${{ matrix.deno-version }}
uses: denoland/setup-deno@v1
with:
Expand Down Expand Up @@ -54,7 +58,7 @@ jobs:
env:
CI: true
run: |
npm ci
npm install
npm run prepack
npm link
Expand All @@ -63,7 +67,7 @@ jobs:
env:
CI: true
run: |
npm ci
npm install
npm run prepack
npm link @nats-io/obj
npm test
8 changes: 6 additions & 2 deletions .github/workflows/services.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,10 @@ jobs:
with:
fetch-depth: 1

- name: Remove NPM Workspace
run: |
rm -f package.json
- name: Use Deno Version ${{ matrix.deno-version }}
uses: denoland/setup-deno@v1
with:
Expand Down Expand Up @@ -64,7 +68,7 @@ jobs:
env:
CI: true
run: |
npm ci
npm install
npm run prepack
npm link
Expand All @@ -73,7 +77,7 @@ jobs:
env:
CI: true
run: |
npm ci
npm install
npm run prepack
npm link @nats-io/services
npm test
8 changes: 6 additions & 2 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,10 @@ jobs:
with:
fetch-depth: 1

- name: Remove NPM Workspace
run: |
rm -f package.json
- name: Use Deno Version ${{ matrix.deno-version }}
uses: denoland/setup-deno@v1
with:
Expand Down Expand Up @@ -55,8 +59,8 @@ jobs:
CI: true
NGS_CI_USER: ${{ secrets.NGS_CI_USER }}
run: |
deno task check-versions --module ${{ matrix.module }}
deno task check-dependencies --module ${{ matrix.module }}
deno run -A bin/check-bundle-version.ts --module ${{ matrix.module }}
deno task check-dependencies
deno task test-${{ matrix.module }}
# - name: Build nats.js
Expand Down
16 changes: 10 additions & 6 deletions .github/workflows/transport-node-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,10 @@ jobs:
with:
fetch-depth: 1

- name: Remove NPM Workspace
run: |
rm -f package.json
- name: Use Deno Version ${{ matrix.deno-version }}
uses: denoland/setup-deno@v1
with:
Expand All @@ -53,7 +57,7 @@ jobs:
env:
CI: true
run: |
npm ci
npm install
npm run prepack
npm link
Expand All @@ -62,7 +66,7 @@ jobs:
env:
CI: true
run: |
npm ci
npm install
npm run prepack
npm link
Expand All @@ -71,7 +75,7 @@ jobs:
env:
CI: true
run: |
npm ci
npm install
npm run prepack
npm link
Expand All @@ -80,7 +84,7 @@ jobs:
env:
CI: true
run: |
npm ci
npm install
npm run prepack
npm link
Expand All @@ -89,7 +93,7 @@ jobs:
env:
CI: true
run: |
npm ci
npm install
npm run prepack
npm link
Expand All @@ -98,7 +102,7 @@ jobs:
env:
CI: true
run: |
npm ci
npm install
npm run prepack
npm link @nats-io/nats-core
npm link @nats-io/jetstream
Expand Down
4 changes: 4 additions & 0 deletions .github/workflows/version_bump.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,10 @@ jobs:
- name: Clone repository
uses: actions/checkout@v4

- name: Remove NPM Workspace
run: |
rm -f package.json
- name: Set up Deno
uses: denoland/setup-deno@v1

Expand Down
25 changes: 8 additions & 17 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1,20 +1,11 @@
.DS_Store
deno.lock
package-lock.json

**/build/
**/lib/
**/node_modules/

debug/
.idea/
.vscode/


# nyc test coverage
benchmark/
coverage/

# ide crud
vcs.xml


lib/
debug/
lib/
build/
package-lock.json
deno.lock
node_modules/
4 changes: 2 additions & 2 deletions GOVERNANCE.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# NATS.deno Client Governance
# NATS.js Governance

NATS.deno Client is part of the NATS project and is subject to the
NATS.js is part of the NATS project and is subject to the
[NATS Governance](https://github.com/nats-io/nats-general/blob/master/GOVERNANCE.md).
10 changes: 0 additions & 10 deletions OWNERS.md

This file was deleted.

4 changes: 0 additions & 4 deletions TODO.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,3 @@
# TODO

- [-] Package nuidjs as its own project
- [ ] Move nats-base-client to its own project
- [-] Transport send batching (current batching seems to perform best)

## BUGS
Loading

0 comments on commit 432bcf2

Please sign in to comment.