Skip to content

Add CODEOWNERS - #1958

Merged
dweindl merged 1 commit into
developfrom
codeowners
Jan 27, 2023
Merged

Add CODEOWNERS#1958
dweindl merged 1 commit into
developfrom
codeowners

Conversation

@dweindl

@dweindl dweindl commented Jan 26, 2023

Copy link
Copy Markdown
Member

Not much to see. This is for auto-assigning reviewers to pull requests.

Not much to see. This is for auto-assigning reviewers to pull requests.
@dweindl
dweindl requested a review from FFroehlich January 26, 2023 11:06
@codecov

codecov Bot commented Jan 26, 2023

Copy link
Copy Markdown

Codecov Report

Merging #1958 (efebdf0) into develop (ed36b2d) will decrease coverage by 0.12%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1958      +/-   ##
===========================================
- Coverage    76.03%   75.92%   -0.12%     
===========================================
  Files           76       76              
  Lines        12982    12982              
===========================================
- Hits          9871     9856      -15     
- Misses        3111     3126      +15     
Flag Coverage Δ
cpp 73.17% <ø> (-0.06%) ⬇️
petab 59.69% <ø> (ø)
python 68.74% <ø> (-0.23%) ⬇️
sbmlsuite ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/exception.cpp 74.28% <0.00%> (-5.72%) ⬇️
...thon/sdist/amici/conserved_quantities_demartino.py 63.60% <0.00%> (-1.96%) ⬇️
src/sundials_matrix_wrapper.cpp 80.61% <0.00%> (-0.41%) ⬇️
src/solver_cvodes.cpp 70.06% <0.00%> (-0.17%) ⬇️

@FFroehlich FFroehlich left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@dweindl
dweindl merged commit b471e0b into develop Jan 27, 2023
@dweindl
dweindl deleted the codeowners branch January 27, 2023 08:14
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.

2 participants