-
-
Notifications
You must be signed in to change notification settings - Fork 65
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add releases to JSR #291
add releases to JSR #291
Conversation
@@ -34,8 +34,8 @@ export class Eta { | |||
renderStringAsync = renderStringAsync; | |||
|
|||
filepathCache: Record<string, string> = {}; | |||
templatesSync = new Cacher<TemplateFunction>({}); | |||
templatesAsync = new Cacher<TemplateFunction>({}); | |||
templatesSync: Cacher<TemplateFunction> = new Cacher<TemplateFunction>({}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These annotations are useless thanks to TypeScript type inference.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, JSR complains though; it won't generate docs for types that take too many cycles to resolve; running the dry-run it said to add explicit types for those interfaces to enable auto-docs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, now I understand 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any speedup is probably imperceptible on these in any non-constrained system, but I figured it wouldn't hurt to add them.
@josh-hemphill thanks so much for implementing this! I just merged and published, so https://jsr.io/@eta-dev/[email protected] is live! |
Awesome! Thanks! |
Adding a step in the release pipeline to update a
jsr.json
when creating a new release version, and then release to JSR if npm release succeeded.Just tried shortening the update scripts, I could just put it inline in the scripts if you prefer:
The only other step to be able to release to JSR would be registering the package scope/name and auth on the first upload.
closes #290