-
-
Notifications
You must be signed in to change notification settings - Fork 43
Support Deno #77
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
Support Deno #77
Conversation
update documentation, good enough? or add documentation in this repo? |
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.
Hi @jp-knj! Sorry we dropped the ball on this — just saw it now while looking through the open PRs. I had a few questions about the approach.
- deno now follows the same pattern as npm/yarn/pnpm/bun - unified install step handles all package managers - support npm dependencies in hybrid Deno projects via deno install - use package-manager input for Deno version (e.g., [email protected])
@delucis, Thanks for spotting that! |
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.
Thanks for the updates @jp-knj!
Co-authored-by: Chris Swithinbank <[email protected]>
Co-authored-by: Chris Swithinbank <[email protected]>
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.
Thanks again for this work @jp-knj!
Summary
Adds first-class Deno support to
withastro/action
, enabling projects thatuse
deno.json
/deno.lock
to rundeno task build
out-of-the-box.Changes
deno.json(c)
ordeno.lock
; setsPACKAGE_MANAGER=deno
deno-version
input (default1.x
)denoland/setup-deno@v2
PACKAGE_MANAGER=deno
Related issue / discussion
Closes #65 (
good first issue
: Deno support)Test plan
pnpm build && act -j test-deno
Ref