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

ci: add depency for librosa #121

Merged
merged 4 commits into from
Feb 12, 2021
Merged

ci: add depency for librosa #121

merged 4 commits into from
Feb 12, 2021

Conversation

deepcharles
Copy link
Owner

A notebook example needs librosa and librosa needs a library that cannot be installed with pip (see here).

We add the dependency to the gh actions that generate the docs.

@github-actions github-actions bot added the Type: CI Changes to CI configuration files and scripts label Feb 12, 2021
@codecov
Copy link

codecov bot commented Feb 12, 2021

Codecov Report

Merging #121 (7bdfe08) into master (68074b7) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #121   +/-   ##
=======================================
  Coverage   95.12%   95.12%           
=======================================
  Files          41       41           
  Lines         965      965           
=======================================
  Hits          918      918           
  Misses         47       47           

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 68074b7...bf1c9fe. Read the comment docs.

@deepcharles deepcharles merged commit 743eb5d into master Feb 12, 2021
@deepcharles deepcharles deleted the ci/build-docs branch February 12, 2021 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: CI Changes to CI configuration files and scripts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant