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: pytest setup fix #2177

Merged
merged 1 commit into from
Aug 1, 2023
Merged

CI: pytest setup fix #2177

merged 1 commit into from
Aug 1, 2023

Conversation

mloubout
Copy link
Contributor

@mloubout mloubout commented Aug 1, 2023

Fixes pytest config issue, setup.cfg is ignored if pytest.ini exsists so our options were being ignored

@mloubout mloubout added the CI continuous integration label Aug 1, 2023
@codecov
Copy link

codecov bot commented Aug 1, 2023

Codecov Report

Merging #2177 (dcb8ea6) into master (6f75a9e) will increase coverage by 0.06%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #2177      +/-   ##
==========================================
+ Coverage   87.03%   87.09%   +0.06%     
==========================================
  Files         223      226       +3     
  Lines       39969    40136     +167     
  Branches     7302     7324      +22     
==========================================
+ Hits        34787    34957     +170     
  Misses       4600     4600              
+ Partials      582      579       -3     
Files Changed Coverage Δ
examples/seismic/elastic/elastic_example.py 77.14% <100.00%> (+28.57%) ⬆️
...mples/seismic/viscoelastic/viscoelastic_example.py 77.14% <100.00%> (+28.57%) ⬆️
tests/test_mpi.py 99.09% <100.00%> (+0.22%) ⬆️

... and 12 files with indirect coverage changes

@mloubout mloubout merged commit d87430f into master Aug 1, 2023
@mloubout mloubout deleted the pytest-fix branch August 1, 2023 23:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant