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

feat(release): Use github actions to release binaries #5

Merged
merged 4 commits into from
Oct 18, 2023
Merged

feat(release): Use github actions to release binaries #5

merged 4 commits into from
Oct 18, 2023

Conversation

pwnwriter
Copy link
Contributor

HelloW, You got a really nice project there. Congrats on that 🥂

This pr will allow you to use github actions and release binaries of presenterm .

Few notes :

  1. Don't forget to add CARGO_TOKEN as secret before merge for crates.io releases

  2. Consider adding a CHANGELOG.md, You can use git-cliff for that. I'm not super familiar with this but still you can get some idea from my repos

  3. Allow github actions to publish


Image ->

screenshot_2023-10-18_10-57-01


The process would go like this . Tested here

screenshot_2023-10-18_11-06-01

  1. I added cargo profiles for release,bench,test,dev. It was helping me a lot to minimize the binary size and test with minimal efforts.

Feel free to make changes <3

@pwnwriter
Copy link
Contributor Author

I have added splashes. Previously, it would do nothing if no arguments were provided. You can see this in the image below.

POC:

Before :

screenshot_2023-10-18_14-22-55

Now :

screenshot_2023-10-18_14-23-16

@pwnwriter
Copy link
Contributor Author

If you have any comment on ascii or design let me know 💯

@pwnwriter
Copy link
Contributor Author

Additionally, if you would like me to work on the readme, just say hi. You can refer to some of my readmes for examples. 🐼

@mfontanini
Copy link
Owner

Awesome, thanks so much! I will add a changelog before next release 👌. Your readme looks nice but I think I'd prefer something a bit more minimalistic.

@mfontanini mfontanini merged commit a93f7ae into mfontanini:master Oct 18, 2023
3 checks passed
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