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

chore: Add noxfile to run pytest across multiple python versions #40

Merged
merged 2 commits into from
Jan 2, 2023

Conversation

Higgcz
Copy link
Contributor

@Higgcz Higgcz commented Nov 23, 2022

This PR adds a configuration script for Nox.

Nox is useful mainly when you finish working on some bigger feature and you want to be sure it works in all supported versions of Python.

In order to use Nox, just install it:

pip install nox

And run it:

nox

Nox will create 4 independent environments - this takes quite a while, so I've set Nox to reuse already built environments, which means if you change something inside an environment, you need to manually remove it from .nox/ directory.

As the next step, we should run all Python versions tests inside Github Actions.

@Higgcz Higgcz added the enhancement New feature or request label Nov 23, 2022
@Higgcz Higgcz self-assigned this Nov 23, 2022
@codecov-commenter
Copy link

codecov-commenter commented Nov 23, 2022

Codecov Report

Merging #40 (b1ae33a) into master (d3e3f0a) will decrease coverage by 0.78%.
The diff coverage is n/a.

❗ Current head b1ae33a differs from pull request most recent head eba10ac. Consider uploading reports for the commit eba10ac to get more accurate results

@@            Coverage Diff             @@
##           master      #40      +/-   ##
==========================================
- Coverage   52.97%   52.19%   -0.79%     
==========================================
  Files          43       43              
  Lines        6233     6246      +13     
==========================================
- Hits         3302     3260      -42     
- Misses       2931     2986      +55     
Impacted Files Coverage Δ
src/evotorch/core.py 61.59% <0.00%> (-3.96%) ⬇️
src/evotorch/operators/base.py 33.01% <0.00%> (+0.30%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@Higgcz Higgcz force-pushed the feature/nox branch 2 times, most recently from b1ae33a to b9b6cde Compare November 25, 2022 18:22
Copy link
Collaborator

@NaturalGradient NaturalGradient left a comment

Choose a reason for hiding this comment

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

I have tried it out and barring the addition of the swig dependency, which is now added, it works perfectly!

@Higgcz Higgcz merged commit 0216aad into master Jan 2, 2023
@Higgcz Higgcz deleted the feature/nox branch January 2, 2023 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants