Skip to content

Conversation

@dweindl
Copy link
Member

@dweindl dweindl commented May 5, 2023

No description provided.

@dweindl dweindl requested a review from a team as a code owner May 5, 2023 20:25
Copy link
Collaborator

@fbergmann fbergmann left a comment

Choose a reason for hiding this comment

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

alright all good with me ... those poor people with py38 will have the last release of petab to be happy with

@fbergmann
Copy link
Collaborator

OMG ... readthedocs does not do py39 yet ...

rtd

told you forcing it to 3.9 might not be a good idea

@codecov-commenter
Copy link

codecov-commenter commented May 5, 2023

Codecov Report

Merging #200 (f8bc2b5) into develop (1964ca1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #200   +/-   ##
========================================
  Coverage    76.25%   76.25%           
========================================
  Files           34       34           
  Lines         3155     3155           
  Branches       764      764           
========================================
  Hits          2406     2406           
  Misses         552      552           
  Partials       197      197           

@dweindl
Copy link
Member Author

dweindl commented May 5, 2023

It does, if we ask nicely. Wait for it...

@dweindl dweindl merged commit 335a8d4 into develop May 5, 2023
@dweindl dweindl deleted the python39 branch May 5, 2023 22:04
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