Skip to content

chore(KNO-9521): add CJS instructions to README - #126

Merged
mattmikolay merged 2 commits into
nextfrom
mattmik-kno-9521-node-sdk-update-readme-to-describe-jose-v5-requirement-for
Aug 20, 2025
Merged

chore(KNO-9521): add CJS instructions to README#126
mattmikolay merged 2 commits into
nextfrom
mattmik-kno-9521-node-sdk-update-readme-to-describe-jose-v5-requirement-for

Conversation

@mattmikolay

Copy link
Copy Markdown
Contributor

This PR updates README.md to include instructions for installing a CJS-compatible version of jose when using a version of Node.js that does not support loading ES modules using require.

@linear

linear Bot commented Aug 20, 2025

Copy link
Copy Markdown

@mattmikolay
mattmikolay changed the base branch from main to next August 20, 2025 13:25
@mattmikolay
mattmikolay marked this pull request as ready for review August 20, 2025 13:27
@mattmikolay
mattmikolay requested a review from a team as a code owner August 20, 2025 13:27
@mattmikolay
mattmikolay requested review from MikeCarbone, kylemcd and thomaswhyyou and removed request for a team and MikeCarbone August 20, 2025 13:27

@kylemcd kylemcd left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice!

@mattmikolay
mattmikolay merged commit a567528 into next Aug 20, 2025
7 checks passed
@mattmikolay
mattmikolay deleted the mattmik-kno-9521-node-sdk-update-readme-to-describe-jose-v5-requirement-for branch August 20, 2025 14:44
This was referenced Aug 21, 2025
meryldakin pushed a commit that referenced this pull request Aug 25, 2025
* feat(mcp): add code execution tool

* chore(KNO-9521): add CJS instructions to README (#126)

* chore: add package to package.json

* chore(client): qualify global Blob

* chore: update CI script

* release: 1.13.0

---------

Co-authored-by: stainless-app[bot] <142633134+stainless-app[bot]@users.noreply.github.com>
Co-authored-by: Matthew Mikolay <mmikolay@knock.app>
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