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

#53 Options deps are NOT optional if it won't run without them #55

Closed

Conversation

factoidforrest
Copy link
Contributor

this is causing failures with certain build systems that expect the packages to be, you know, optional.

this is causing failures with certain build systems that expect the packages to be, you know, optional.
@codecov-io
Copy link

Codecov Report

Merging #55 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #55   +/-   ##
=======================================
  Coverage   86.37%   86.37%           
=======================================
  Files          24       24           
  Lines        3750     3750           
=======================================
  Hits         3239     3239           
  Misses        511      511

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1088992...9c57dd9. Read the comment docs.

@bahamat
Copy link
Member

bahamat commented Jan 3, 2024

Fixed in 026ef47.

@bahamat bahamat closed this Jan 3, 2024
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.

3 participants