Skip to content

Conversation

@alexolivier
Copy link
Contributor

@alexolivier alexolivier commented Nov 17, 2025

Following @cerbos/cerbos-sdk-javascript

Signed-off-by: Alex Olivier <[email protected]>
Signed-off-by: Alex Olivier <[email protected]>
Signed-off-by: Alex Olivier <[email protected]>
@alexolivier alexolivier marked this pull request as ready for review November 17, 2025 09:54
Signed-off-by: Alex Olivier <[email protected]>
@alexolivier alexolivier changed the title deprecate older versions of node and mongo major ver: deprecate older versions of node and mongo Nov 17, 2025
@alexolivier alexolivier changed the title major ver: deprecate older versions of node and mongo major ver: Deprecate older versions of node and mongoose Nov 17, 2025
Signed-off-by: Alex Olivier <[email protected]>
@alexolivier alexolivier requested a review from haines November 17, 2025 13:55
Signed-off-by: Alex Olivier <[email protected]>
],
"sideEffects": false,
"publishConfig": {
"access": "public"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It might be worth setting up publication from GitHub Actions on tag push (it's quite easy now that they support OIDC), in which case you can chuck "provenance": true in here as well and get a nice green tick on the npm page.

Co-authored-by: Andrew Haines <[email protected]>
Signed-off-by: Alex Olivier <[email protected]>
alexolivier and others added 3 commits November 17, 2025 15:52
Co-authored-by: Andrew Haines <[email protected]>
Signed-off-by: Alex Olivier <[email protected]>
Signed-off-by: Alex Olivier <[email protected]>
Signed-off-by: Alex Olivier <[email protected]>
@alexolivier alexolivier merged commit 1be0bad into main Nov 17, 2025
13 checks passed
@alexolivier alexolivier deleted the deprecate-node-mongo branch November 17, 2025 16:13
- uses: actions/setup-node@v4
with:
node-version: "20"
registry-url: "https://registry.npmjs.org"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is, annoyingly, wrong in the docs: npm/documentation#1748

Suggested change
registry-url: "https://registry.npmjs.org"

- uses: actions/setup-node@v4
with:
node-version: "20"
registry-url: "https://registry.npmjs.org"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
registry-url: "https://registry.npmjs.org"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants