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

Remove unused dependencies #85

Merged
merged 1 commit into from
Sep 9, 2021
Merged

Remove unused dependencies #85

merged 1 commit into from
Sep 9, 2021

Conversation

hendrikmaus
Copy link
Owner

@hendrikmaus hendrikmaus commented Sep 9, 2021 •
edited
Loading

https://github.com/est31/cargo-udeps reports "thiserror" as unused by all targets:

$ cargo +nightly udeps --all-targets
unused dependencies:
`helm-templexer v2.0.0 (/home/hmaus/Development/helm-templexer)`
└─── dependencies
     └─── "thiserror"

Remove unused dependencies
5302721 
https://github.com/est31/cargo-udeps reports "thiserror" as unused
by all targets:

```shell
$ cargo +nightly udeps --all-targets
unused dependencies:
`helm-templexer v2.0.0 (/home/hmaus/Development/helm-templexer)`
└─── dependencies
     └─── "thiserror"
```
@hendrikmaus hendrikmaus added the internal Internal changes like release process, CI etc. label Sep 9, 2021
@hendrikmaus hendrikmaus self-assigned this Sep 9, 2021
@hendrikmaus hendrikmaus added this to In progress in v2 via automation Sep 9, 2021
@hendrikmaus hendrikmaus merged commit 7020798 into master Sep 9, 2021
@hendrikmaus hendrikmaus deleted the remove-unused-dependencies branch September 9, 2021 12:04
v2 automation moved this from In progress to Done Sep 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AlexanderThaller AlexanderThaller Awaiting requested review from AlexanderThaller AlexanderThaller is a code owner

@Kenec Kenec Awaiting requested review from Kenec Kenec is a code owner

Assignees

@hendrikmaus hendrikmaus

Labels
internal Internal changes like release process, CI etc.
Projects
v2
Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@hendrikmaus