Skip to content

wip: publish to jsr#321

Open
peterhirn wants to merge 1 commit intomainfrom
feat/jsr
Open

wip: publish to jsr#321
peterhirn wants to merge 1 commit intomainfrom
feat/jsr

Conversation

@peterhirn
Copy link
Contributor

@peterhirn peterhirn commented Apr 29, 2025

@AugustinMauroy
Copy link

In theory it's should be publishable. I'm going to test on a local JSR instance.

@@ -0,0 +1,14 @@
{
"name": "@peterhirn/test",

Choose a reason for hiding this comment

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

Suggested change
"name": "@peterhirn/test",
"name": "@phi-ag/argon2",

Why not having the samething that npm ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, this was only used for testing.

@AugustinMauroy
Copy link

on readme you can add JSR badge
https://jsr.io/docs/badges#version-badge

@peterhirn
Copy link
Contributor Author

Last time I checked it didn't work, so I opened the issue mentioned in the first comment.

In the meantime I added the PURE_WASI flag to the build (see f0c7d0d) which removed all imports, so it might work now. Will check later.

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