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

build: switch to stable qrules and ampform #265

Merged
merged 7 commits into from
Apr 19, 2021

Conversation

redeboer
Copy link
Member

@redeboer redeboer commented Apr 19, 2021

In addition, extract pytest configuration to pytest.ini and define testpaths there. Since b9b72ed, it is become impossible to run tests through VSCode. Defining testpaths fixes this. In addition, it removes the last 'non-tox' section from the tox.ini file (see 7eb3866)

@redeboer redeboer added the 🖱️ DX Improvements to the Developer Experience label Apr 19, 2021
@redeboer redeboer changed the title ci: upgrade developer dependencies fix: define testpaths in pytest.ini Apr 19, 2021
@redeboer redeboer added the 🐛 Bug Something isn't working label Apr 19, 2021
@redeboer redeboer self-assigned this Apr 19, 2021
@redeboer redeboer force-pushed the create-pull-request/patch-1618798888 branch from 7e38a41 to f60fca9 Compare April 19, 2021 07:24
@redeboer redeboer force-pushed the create-pull-request/patch-1618798888 branch from f60fca9 to 9f489f1 Compare April 19, 2021 07:27
@codecov
Copy link

codecov bot commented Apr 19, 2021

Codecov Report

Merging #265 (b802d1c) into main (5fde3be) will increase coverage by 84.19%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           main     #265       +/-   ##
=========================================
+ Coverage      0   84.19%   +84.19%     
=========================================
  Files         0       13       +13     
  Lines         0      753      +753     
=========================================
+ Hits          0      634      +634     
- Misses        0      119      +119     
Flag Coverage Δ
unittests 84.19% <ø> (?)

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

Impacted Files Coverage Δ
src/tensorwaves/optimizer/minuit.py 93.22% <0.00%> (ø)
src/tensorwaves/physics.py 78.26% <0.00%> (ø)
src/tensorwaves/interfaces.py 82.14% <0.00%> (ø)
src/tensorwaves/optimizer/_parameter.py 100.00% <0.00%> (ø)
src/tensorwaves/__init__.py 100.00% <0.00%> (ø)
src/tensorwaves/optimizer/__init__.py 71.42% <0.00%> (ø)
src/tensorwaves/data/__init__.py 96.77% <0.00%> (ø)
src/tensorwaves/data/transform.py 100.00% <0.00%> (ø)
src/tensorwaves/optimizer/callbacks.py 74.72% <0.00%> (ø)
src/tensorwaves/model.py 83.33% <0.00%> (ø)
... and 3 more

@redeboer redeboer changed the title fix: define testpaths in pytest.ini build: switch to qrules and ampform stable releases Apr 19, 2021
@redeboer redeboer added 🔨 Maintenance Maintenance and upkeep improvements 🖱️ DX Improvements to the Developer Experience and removed 🐛 Bug Something isn't working 🖱️ DX Improvements to the Developer Experience labels Apr 19, 2021
@redeboer redeboer changed the title build: switch to qrules and ampform stable releases build: switch to stable qrules and ampform Apr 19, 2021
@redeboer redeboer merged commit bbb9294 into main Apr 19, 2021
@redeboer redeboer deleted the create-pull-request/patch-1618798888 branch April 19, 2021 07:59
@redeboer redeboer added this to the Release 0.2.4 milestone May 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🖱️ DX Improvements to the Developer Experience 🔨 Maintenance Maintenance and upkeep improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants