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

build: make dependency-check script more verbose #1116

Closed
wants to merge 1 commit into from

Conversation

chadoh
Copy link
Contributor

@chadoh chadoh commented Dec 7, 2023

What

Add more printf and echo statements to the dependency-check script so we can figure out where it fails.

Why

Right now if something goes wrong it's really hard to tell what.

Known limitations

This branch should probably also fix whatever is causing the dependency-check script to fail on CI.

@sreuland
Copy link
Contributor

@chadoh , this seems closely aligned to #1110, do you want to take that issue and add Closes #1110 here?

@chadoh
Copy link
Contributor Author

chadoh commented Dec 11, 2023

@leighmcculloch mentioned in Slack that we need to remove this custom script and use cargo-deny as in other repos; I think he was working on that?

@chadoh chadoh closed this Dec 11, 2023
@leighmcculloch
Copy link
Member

I'm not working on it 😄.

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.

4 participants