Skip to content
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 compile action #20

Closed
wants to merge 3 commits into from
Closed

Add compile action #20

wants to merge 3 commits into from

Conversation

404Wolf
Copy link
Collaborator

@404Wolf 404Wolf commented Mar 10, 2025

No description provided.

@maxmcd
Copy link
Member

maxmcd commented Mar 11, 2025

I think we want people installing with deno install -A --name vt https://val.town/v/std/vt (or similar). I think it would be nice to avoid maintaining these? Maybe I am missing a benefit?

@404Wolf
Copy link
Collaborator Author

404Wolf commented Mar 12, 2025

I think we want people installing with deno install -A --name vt https://val.town/v/std/vt (or similar). I think it would be nice to avoid maintaining these? Maybe I am missing a benefit?

Didn't know about deno install! I like avoiding -A even though it probably doesn't really matter. I don't know how deno install works so I'll look into it soon, but I know that there's decent performance improvement with deno compiled binaries, and you remove the deno dependency entirely (which maybe doesn't matter, curious to hear what you think). Also, it will be useful if we want to package this on other package managers (brew install vt?). It's nice that the entire project can be distributed as a single file.

@maxmcd
Copy link
Member

maxmcd commented Mar 12, 2025

Part of the reason I was really drawn to the deno install workflow is that we're downstream of the deno cli so we're helping push people to install it.

Also having https://val.town in there helps broadcast where the code is hosted and that you can also create installable things on val.town yourself.

We can change -A to whatever permissions we think we need.

@404Wolf
Copy link
Collaborator Author

404Wolf commented Mar 13, 2025

Okay, yeah I agree, I'll close this

@404Wolf 404Wolf closed this Mar 13, 2025
@404Wolf 404Wolf deleted the builds branch April 7, 2025 22:05
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