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

Split cargo-deadlinks and deadlinks into two binaries #87

Merged
merged 3 commits into from
Nov 18, 2020

Conversation

jyn514
Copy link
Contributor

@jyn514 jyn514 commented Oct 30, 2020

  • Expose public walk_dir function, which prints all missing files
  • impl Clone for CheckContext
  • Put cargo dependencies behind a feature gate
  • Switch from Into to From to help type inference

Closes #47.

r? @hobofan - what do you think? Does it make sense to expose walk_dir publicly? If not, I can move it into mod shared.

@jyn514 jyn514 added the C-enhancement Category: This is a new feature label Oct 30, 2020
@jyn514 jyn514 force-pushed the deadlinks-bin branch 2 times, most recently from b875b2c to 154ed75 Compare October 30, 2020 21:26
@jyn514 jyn514 added the S-waiting-on-review Status: Waiting for either a maintainer or issue author to review label Nov 12, 2020
@hobofan
Copy link
Member

hobofan commented Nov 14, 2020

@jyn514 Yeah, I think walk_dir should be exposed publicly (which I guess is also the main point of using it as a library).

@hobofan hobofan removed the S-waiting-on-review Status: Waiting for either a maintainer or issue author to review label Nov 14, 2020
@jyn514 jyn514 force-pushed the deadlinks-bin branch 2 times, most recently from bb21d3e to 08ebaf8 Compare November 18, 2020 01:19
- Expose public `walk_dir` function, which prints all missing files
- `impl Clone for CheckContext`
- Put cargo dependencies behind a feature gate
- Switch from `Into` to `From` to help type inference
This shouldn't affect the end-user at all, it just makes things easier
for development.
@jyn514 jyn514 merged commit d27e9f4 into deadlinks:master Nov 18, 2020
@jyn514 jyn514 deleted the deadlinks-bin branch November 18, 2020 01:47
@jyn514 jyn514 mentioned this pull request Nov 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: This is a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Provide a deadlinks crate
2 participants