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.

Sign up for GitHub

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

Jump to bottom

Cleaner executable name #3

Closed
merwok opened this issue Nov 19, 2017 · 2 comments
Closed

Cleaner executable name #3

merwok opened this issue Nov 19, 2017 · 2 comments
Assignees
@dpmcmlxxvi
Labels
enhancement

Comments

@merwok
Copy link
Contributor

merwok commented Nov 19, 2017

Hello! Thanks for this program, I was checking out qstats and ministat and this one seems more useful.

I was surprised to see that the compiled executable is named clistats.exe; this seems very uncommon for a unix program. Could this become clistats?

In addition, it should be added to .gitignore.
@dpmcmlxxvi dpmcmlxxvi self-assigned this Nov 19, 2017
@dpmcmlxxvi
Copy link
Owner

dpmcmlxxvi commented Nov 19, 2017 •
edited
Loading

Sure thing. Easy enough fix. Will look into it when I get a few free cycles.

merwok added a commit to merwok-forks/clistats that referenced this issue Nov 19, 2017
@merwok
change executable name (dpmcmlxxvi#3)
fb4b130
@merwok merwok mentioned this issue Nov 19, 2017
Merged
@merwok
Copy link
Contributor Author

merwok commented Nov 19, 2017

Sent a PR! The examples in README already used clistats so things seem good.

dpmcmlxxvi added a commit that referenced this issue Nov 20, 2017
@dpmcmlxxvi
Merge pull request #4 from merwok-forks/master
395cd00 
Change executable name (#3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dpmcmlxxvi dpmcmlxxvi

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@merwok @dpmcmlxxvi