search

8go / matrix-commander-rs

simple but convenient CLI-based Matrix client app for sending and receiving (in Rust)
GNU General Public License v3.0
79 stars 2 forks source link

Create GitHub release #1

Closed hwittenborn closed 1 year ago

hwittenborn commented 1 year ago

Hi! I just saw this project after creating a similar project that was basically gonna be matrix-commander in Rust, but then I saw this and I didn't see much point in it anymore.

Anyway, I definitely would prefer to just use this, but I'd like to package it up first, and doing so would require a release to download from. Would it be possible to create a release so I could do such?

8go commented 1 year ago

Hi @hwittenborn

You can always contribute to matrix-commander-rs. There is plenty to do! Feel free to add your desired features by creating PRs.

matrix-commander-rs is listed on https://crates.io/crates/matrix-commander and here (Github).

Standard Rust procedure:

A task item on the To-Do list is: create a GitHub action so that on each git push it creates a release automatically, e.g. for a popular platform like Ubuntu. If you have experience with Github actions or want to learn something new, try to create such an action. You can do that for a Rust project in your Github account and once it works we adapt it and make a PR here.

Just some ideas :smile:

hwittenborn commented 1 year ago

Hey! I'd be down with implementing an automated action, but at least for the time being could a manual release be made here? I've been trying to package this up and it'd just be nicer if there could be a release now and then needed automation could be set up for future releases.

8go commented 1 year ago

As exception I can do one and only one manual release here. But this is definitely not the way forward, automation is key to everything. Also helps with reproducibility, consistency, ...

8go commented 1 year ago

What are your planning to do with the release?

Is it for publication on https://mpr.makedeb.org?

hwittenborn commented 1 year ago

Is it for publication on https://mpr.makedeb.org?

Yeah that's it. I plan on installing it on a server so I can monitor various things on it and be notified when things aren't working right.

hwittenborn commented 1 year ago

But this is definitely not the way forward, automation is key to everything.

Definitely agree, I try to always automate where feasible too. At least in git repos it just makes things easier, all I gotta do is push and CI does the rest. Definitely love the approach.

8go commented 1 year ago

Github action workflow has been written. In the future releases will show up automatically.

hwittenborn commented 1 year ago

Thanks for getting that written up! I'll have the MPR package up sometime soon too

8go commented 1 year ago

Let me know when the MPR package is in place :) :+1:

hwittenborn commented 1 year ago

Yo sorry for the delay in the MPR package, I've been busy with another project at the moment and I haven't gotten around to getting this packaged yet. I do want to still, I just wanted to give a heads up that I've just been doing some other stuff right now.

8go commented 1 year ago

@hwittenborn No hurry, take your time!

hwittenborn commented 1 year ago

I'm getting this packaged up right now @8go - are you fine if I name the Rust version of the package matrix-commander, or do you want it to be matrix-commander-rs? I want the first, but I don't know how feature complete this Rust version currently is and if you'd just want otherwise or anything.

8go commented 1 year ago

Are you only packaging Rust applications? If so, matrix-commander is fine. Then there is no confusion, if all is Rust.

If you are packaging all sorts of apps, Python, Rust, Java, etc. then I think it is better to use name matrix-commander-rs (so that you can later package the Python version and so that people understand the difference).

The Py and Rust version are NOT equal, Py has more features.

hwittenborn commented 1 year ago

Got it - yeah I package a bunch of different stuff in different languages, so I'll do matrix-commander-rs then. I'll send a message when it's all done.

8go commented 1 year ago

Yes, I think in this case as you said, matrix-commander-rs is better.

hwittenborn commented 1 year ago

Sorry again for the delay @8go, I've been busy with that side project again and it started taking up a bunch of my time again :P. I finally got it packaged up on the MPR though - would you be open to a PR to add instructions for installing it?

8go commented 1 year ago

would you be open to a PR to add instructions for installing it?

Yes, @hwittenborn please.