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

Make clap an optional dependency #69

Merged
merged 1 commit into from
May 7, 2023
Merged

Conversation

wildwestrom
Copy link
Contributor

I'd like to make clap an optional dependency if I just want to install the library, while also keeping the ability to use cargo install to install the binary.

README.md Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented May 7, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (99f0814) 72.72% compared to head (6e523af) 72.72%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #69   +/-   ##
=======================================
  Coverage   72.72%   72.72%           
=======================================
  Files           3        3           
  Lines         176      176           
=======================================
  Hits          128      128           
  Misses         48       48           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@wildwestrom wildwestrom changed the title Make clap and optional dependency Make clap an optional dependency May 7, 2023
Or put in your `Cargo.toml` file:

```toml
tzf-rs = { version = "0.4.0", default-features = false }
Copy link
Owner

Choose a reason for hiding this comment

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

I will try this in tzfpy, and will merge this PR if everything is fine:

tzf-rs = { git =  "https://github.com/wildwestrom/tzf-rs", rev = "6e523af", default-features = false}

Copy link
Owner

@ringsaturn ringsaturn left a comment

Choose a reason for hiding this comment

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

LGTM

@ringsaturn ringsaturn merged commit d8480df into ringsaturn:main May 7, 2023
3 checks passed
@wildwestrom
Copy link
Contributor Author

Awesome, should I wait for the next semver version to see these changes?

@ringsaturn
Copy link
Owner

ringsaturn commented May 7, 2023

Awesome, should I wait for the next semver version to see these changes?

I have release v0.4.1 with this PR.

It has been deployed to https://crates.io/crates/tzf-rs/0.4.1

@wildwestrom
Copy link
Contributor Author

Thank you very much.

@wildwestrom
Copy link
Contributor Author

The readme should be updated to reflect the new version as well.

@ringsaturn
Copy link
Owner

The readme should be updated to reflect the new version as well.

Updated.

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