-
Notifications
You must be signed in to change notification settings - Fork 48
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
Release #12
Comments
What do you mean by a release? I admit I didn't keep up on this project so I did not tag the latest changes. That has been fixed and I've tagged 1.1.0. Was there more that you needed beyond a tag in the repository? If so, perhaps you have an example that you'd like to point me towards so I can see a working example? |
No activity, so I hope that I addressed this concern. |
Thanks, they wouldn't take the PR without tests and I couldn't figure out how to write a test for this. |
Maybe you could look at how tests are written for ncurses, dialog, ttygif,
or other software that simply displays stuff in the console? I am happy to
add tests if that helps get the script adopted in places.
…On Tue, Jan 15, 2019, 01:30 Drew J. Sonne ***@***.*** wrote:
Thanks, they wouldn't take the PR without tests and I couldn't figure out
how to write a test for this.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#12 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAaJPLzkTNbOqMOtK96bfO2xamvYABjDks5vDYOLgaJpZM4UygkS>
.
|
Hi, I'm trying to get your package into homebrew, is it possible to do a release?
The text was updated successfully, but these errors were encountered: