Skip to content

Commit 02f92ac

Browse files
authored
feat: pnpm Workspace (#64)
# Description <!-- Include a summary of the change made and also list the dependencies that are required if any --> Moves the project to pnpm workspaces for better monorepo support. This also addresses the issue of managing the netlify site to avoid any issues with deploying the demo project. ## Type of change <!-- Please select all options that are applicable. --> - [ ] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) - [ ] This change requires a documentation update # Checklist <!-- These must all be followed and checked. --> - [ ] I have followed the contributing guidelines of this project as mentioned in [CONTRIBUTING.md](/CONTRIBUTING.md) - [ ] I have created an [issue](https://github.com/colbyfayock/netlify-plugin-cloudinary/issues) ticket for this PR - [ ] I have checked to ensure there aren't other open [Pull Requests](https://github.com/colbyfayock/netlify-plugin-cloudinary/pulls) for the same update/change? - [ ] I have performed a self-review of my own code - [ ] I have run tests locally to ensure they all pass - [ ] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes needed to the documentation
1 parent 2eedbea commit 02f92ac

31 files changed

+7734
-18079
lines changed

.github/workflows/test_release.yml

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,11 +11,15 @@ jobs:
1111
node: [ '16', '18', '20' ]
1212
steps:
1313
- uses: actions/checkout@v2
14-
- uses: actions/setup-node@v2
14+
- uses: pnpm/action-setup@v2
15+
with:
16+
version: 8
17+
- uses: actions/setup-node@v3
1518
with:
1619
node-version: ${{ matrix.node }}
17-
- run: npm ci
18-
- run: npm run test
20+
cache: 'pnpm'
21+
- run: pnpm install --frozen-lockfile
22+
- run: pnpm test
1923
env:
2024
CLOUDINARY_CLOUD_NAME: ${{ secrets.CLOUDINARY_CLOUD_NAME }}
2125
CLOUDINARY_API_KEY: ${{ secrets.CLOUDINARY_API_KEY }}
@@ -28,12 +32,16 @@ jobs:
2832
runs-on: ubuntu-latest
2933
steps:
3034
- uses: actions/checkout@v2
35+
- uses: pnpm/action-setup@v2
3136
with:
32-
fetch-depth: 0
33-
- uses: actions/setup-node@v2
37+
version: 8
38+
- uses: actions/setup-node@v3
3439
with:
3540
node-version: '18'
36-
- run: npm ci
41+
cache: 'pnpm'
42+
# https://github.com/pnpm/pnpm/issues/3141
43+
registry-url: 'https://registry.npmjs.org'
44+
- run: pnpm install --frozen-lockfile
3745
# Do not store semantic release dependencies in package.json to avoid lower node versions from failing tests
3846
- run: npm install @semantic-release/changelog @semantic-release/git semantic-release
3947
- run: npx semantic-release --branches main

.npmignore

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

.releaserc

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,12 +39,17 @@
3939
"changelogFile": "CHANGELOG.md"
4040
}
4141
],
42-
"@semantic-release/npm",
42+
[
43+
"@semantic-release/npm",
44+
{
45+
"pkgRoot": "netlify-plugin-cloudinary"
46+
}
47+
],
4348
[
4449
"@semantic-release/git",
4550
{
4651
"assets": [
47-
"package.json",
52+
"netlify-plugin-cloudinary/package.json",
4853
"CHANGELOG.md"
4954
]
5055
}

0 commit comments

Comments
 (0)