Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

build: add utils to source distribution #290

Merged
merged 1 commit into from
Oct 7, 2021

Conversation

alexander-held
Copy link
Member

Utilities for producing example ntuples (and histograms, upcoming in #289) previously were available in util/. This folder is renamed to utils/ in preparation for the addition of a histogram-creating utility, and added to MANIFEST.in to make these utilities available in source distributions (as they are used in integration testing).

* added utilities required by integration test to source distribution
* renamed utility folder from util to utils

@codecov
Copy link

codecov bot commented Oct 7, 2021

Codecov Report

Merging #290 (afff557) into master (054c2ef) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #290   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           19        19           
  Lines         1808      1808           
  Branches       283       283           
=========================================
  Hits          1808      1808           

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 054c2ef...afff557. Read the comment docs.

@alexander-held alexander-held merged commit 2b9edf8 into master Oct 7, 2021
@alexander-held alexander-held deleted the build/add-utils-to-sdist branch October 7, 2021 17:34
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant