Skip to content

Add compile action#20

Closed
404Wolf wants to merge 3 commits intomainfrom
builds
Closed

Add compile action#20
404Wolf wants to merge 3 commits intomainfrom
builds

Conversation

@404Wolf
Copy link
Copy Markdown
Contributor

@404Wolf 404Wolf commented Mar 10, 2025

No description provided.

@maxmcd
Copy link
Copy Markdown
Contributor

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
Copy Markdown
Contributor 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
Copy Markdown
Contributor

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
Copy Markdown
Contributor 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