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

Fix warnings #1033

Merged
merged 15 commits into from
Dec 22, 2023
Merged

Fix warnings #1033

merged 15 commits into from
Dec 22, 2023

Conversation

p-snft
Copy link
Member

@p-snft p-snft commented Dec 22, 2023

  • Move to latest revision of GitHub actions
  • Update to current setuptools
  • Update to current pytest API
  • Fix warnings caused by tests using old solph API
  • Silence warnings using in tests for experimental features

This also adds explicit tests for the warnings that are now silenced.

The old versions are no longer suported.
@p-snft p-snft changed the title Update to recent APIs of dependencies Update to recent APIs of build tools Dec 22, 2023
Editable builds using pyroject.toml were introduced with PEP-660.
It is supported since setuptools 64. It was still possible
using legacy mode (setup.py) but led to warnings.
@p-snft p-snft changed the title Update to recent APIs of build tools Fix warnings Dec 22, 2023
FutureWarnings should be eliminated now, ExperimentalFeatureWarnings
and SuspiciousUsageWarnings remain as we test experimental features.
@pep8speaks
Copy link

pep8speaks commented Dec 22, 2023

Hello @p-snft! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2023-12-22 11:36:26 UTC

@p-snft p-snft self-assigned this Dec 22, 2023
@p-snft p-snft marked this pull request as ready for review December 22, 2023 11:31
@p-snft p-snft merged commit 4dd5a76 into dev Dec 22, 2023
14 checks passed
@p-snft p-snft deleted the revision/fix_warnings branch December 22, 2023 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants