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

Remove non-package-related files, update links #41

Merged
merged 10 commits into from
Mar 19, 2021
Merged

Remove non-package-related files, update links #41

merged 10 commits into from
Mar 19, 2021

Conversation

dweindl
Copy link
Member

@dweindl dweindl commented Mar 18, 2021

Remove everything that is not related to the petab Python package. Closes #39

@codecov-io
Copy link

codecov-io commented Mar 18, 2021

Codecov Report

❗ No coverage uploaded for pull request base (develop@b000937). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             develop      #41   +/-   ##
==========================================
  Coverage           ?   78.43%           
==========================================
  Files              ?       24           
  Lines              ?     2323           
  Branches           ?      549           
==========================================
  Hits               ?     1822           
  Misses             ?      364           
  Partials           ?      137           

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 b000937...437b548. Read the comment docs.

@dweindl dweindl requested a review from yannikschaelte March 18, 2021 22:14
@@ -8,7 +8,7 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
python-version: [3.8]
python-version: [3.7, 3.8, 3.9]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3.7 + 3.9 should suffice, no need to test all intermediate versions usually

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since tests here are pretty fast, I'd prefer going for all three.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

LICENSE Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
dweindl and others added 2 commits March 19, 2021 10:26
Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com>
@dweindl dweindl merged commit fd34613 into develop Mar 19, 2021
@dweindl dweindl deleted the split branch March 19, 2021 09:35
@dweindl dweindl mentioned this pull request Mar 19, 2021
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