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

Add climake #901

Closed
wants to merge 5 commits into from
Closed

Add climake #901

wants to merge 5 commits into from

Conversation

Owez
Copy link
Contributor

@Owez Owez commented Jun 2, 2020

https://gitlab.com/Owez/climake now https://github.com/owez/climake
https://crates.io/crates/climake

Dependency-free argument parser only on stdlib, just became production ready and is fully doc'd 😄

Owez added 2 commits June 2, 2020 08:46
Forgot to add in alphabetical order, my apologies
README.md Outdated Show resolved Hide resolved
@palfrey
Copy link
Collaborator

palfrey commented Jul 4, 2020

This has now been open for a month without the errors being resolved. I'm closing this for the moment, but please re-open once they're resolved.

@palfrey palfrey closed this Jul 4, 2020
@Owez
Copy link
Contributor Author

Owez commented Jul 10, 2020

@palfrey Resolved, apologies

@palfrey palfrey reopened this Jul 10, 2020
@palfrey
Copy link
Collaborator

palfrey commented Jul 10, 2020

@palfrey Resolved, apologies

I'm not seeing any new commits here?

Apologies, it's appeared now I've reopened this :)

@palfrey
Copy link
Collaborator

palfrey commented Jul 10, 2020

I'm seeing testing errors however. Any thoughts on this? This makes me somewhat concerned about the quality of the library.

@Owez
Copy link
Contributor Author

Owez commented Jul 10, 2020

Recently moved repository from gitlab to github, changed commit to reflect that & added (now passing) CI. Looks like that failing test was from a last minute doc upgrade before switching

@palfrey
Copy link
Collaborator

palfrey commented Jul 10, 2020

Recently moved repository from gitlab to github, changed commit to reflect that & added (now passing) CI. Looks like that failing test was from a last minute doc upgrade before switching

Coll

@palfrey palfrey closed this Jul 10, 2020
@Owez
Copy link
Contributor Author

Owez commented Jul 10, 2020

Why the close? I added another commit to update remote 4 hours ago, 8e50023

@palfrey
Copy link
Collaborator

palfrey commented Jul 10, 2020

Sorry, I was doing this while doing many other things, and must have somehow accidentally closed. Reopening.

@palfrey palfrey reopened this Jul 10, 2020
README.md Outdated
@@ -754,6 +754,7 @@ See also [About Rust’s Machine Learning Community](https://medium.com/@autumn_

* Argument parsing
* [clap-rs](https://github.com/clap-rs/clap) [[clap](https://crates.io/crates/clap)] — A simple to use, full featured command-line argument parser [<img src="https://api.travis-ci.org/clap-rs/clap.svg?branch=master">](https://travis-ci.org/clap-rs/clap)
* [owez/climake](https://github.com/Owez/climake) [[climake](https://crates.io/crates/climake)] - A simple, dependency-free cli framework
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now there's a CI on that, can you add the build badge please? Sorry about all the stuff here, but this should be it to get it in. Please note that Github Actions badges should specifically reference their main branch (master in this case) to avoid potential cases where the build badge follows some other branch.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No problem, my fault for forgetting

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Owez
Copy link
Contributor Author

Owez commented Jul 15, 2020

@palfrey Added badge ^^

@palfrey
Copy link
Collaborator

palfrey commented Jul 24, 2020

@palfrey
Copy link
Collaborator

palfrey commented Aug 27, 2020

It's been over a month since the last comment, so closing for the moment.

@palfrey palfrey closed this Aug 27, 2020
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.

None yet

2 participants