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

Update poetry.lock and pyproject.toml, add CI check on poetry.lock #72

Merged
merged 4 commits into from
Sep 25, 2023

Conversation

marcofavoritobi
Copy link
Contributor

@marcofavoritobi marcofavoritobi commented Sep 22, 2023

This PR simply refreshes the dependencies by running poetry update. Then, the dependency sections of tox.ini are synchronized with the new content of poetry.lock.

Also pyproject.toml is synchronized with the new poetry.lock, although the update required a couple of cyclic iteration with poetry lock until convergence.

The new version of numpy had some breaking changes which affected the correct behaviour of the library. The errors have been fixed 3eb5350.

Moreover, we added a CI step that checks whether poetry.lock is up-to-date.

@codecov-commenter
Copy link

codecov-commenter commented Sep 22, 2023

Codecov Report

Merging #72 (f64b313) into main (4f1e08e) will not change coverage.
The diff coverage is 100.00%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #72   +/-   ##
=======================================
  Coverage   89.22%   89.22%           
=======================================
  Files          42       42           
  Lines        1782     1782           
=======================================
  Hits         1590     1590           
  Misses        192      192           
Files Changed Coverage Δ
black_it/loss_functions/msm.py 85.07% <100.00%> (ø)
black_it/samplers/halton.py 97.46% <100.00%> (ø)

@marcofavoritobi marcofavoritobi changed the title Update poetry.lock and pyproject.toml Update poetry.lock and pyproject.toml, add CI check on poetry.lock Sep 22, 2023
Copy link
Contributor

@AldoGl AldoGl left a comment

Choose a reason for hiding this comment

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

Needed update, thanks

We also update the deps sections in tox.ini so to make the testenvs synchronized with poetry.lock.
Notable packages whose version specifier has changed are:
- scikit-learn: from ^1.2.2 to ^1.3.1
- joblib: from ^1.2.0 to ^1.3.2
This commit adds a PHONY Makefile target 'poetry-lock-check' that runs the command 'poetry check --lock'.

This target is then used as first step in the CI lint workflow.
Makefile Outdated Show resolved Hide resolved
@marcofavoritobi marcofavoritobi merged commit 507b32a into main Sep 25, 2023
10 checks passed
@marcofavoritobi marcofavoritobi deleted the update-poetry-lock branch September 25, 2023 07:15
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