-
Notifications
You must be signed in to change notification settings - Fork 18
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
feat: support for cargo binstall #58
Conversation
this lets users install the binary directly instead of compiling it themselves or grabbing the releases manually
This is awesome! Note that on windows, moxide is packaged as a zip. Does pkg-fmt account for this? (I guess is tgz related to tar?) |
Also we'd want to include this in the installation section of the readme. leave a like if you want me to add |
Ah, yes, removing the |
update: add to installtion section of the readme
Does it work now? I don't have access to a Windows machine :) |
I don't either 😀 |
The cargo binstall command is not working because markdown-oxide is not on crates.io yet. We could add it, or (I am currently) try to specify the git version in binstall |
BTW I'm currently publishing to crates.io I'm struggling a little because tower-lsp is from a git source! |
Oh right! I was running it locally using the
Nice!
Yeah, that is a little tricky! You could have a look at these |
Is there a way to simply modify the cargo install command to install from git? I was trying to add the --git flag, but it was not working (IO errors?) This way it can be added now, and crates.io publishing can be added later. |
Afaik, you should be able to do that. |
I'm getting this
I'm on nixos as well |
This is what i get:
I am really not sure what could be going wrong here :/ |
Just checking: does |
Nice! |
This lets users install the binary directly instead of manually compiling it or grabbing the releases.
PS: You will have to update the version in the
Cargo.toml
file for new releases from now on!