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: enable -Werror for pytest #518

Merged
merged 10 commits into from
Nov 5, 2023
Merged

CI: enable -Werror for pytest #518

merged 10 commits into from
Nov 5, 2023

Conversation

odesenfans
Copy link
Contributor

No description provided.

Problem: lots of warnings are ignored.

Solution: force developers to care by transforming them into errors.
Modified the storage tests to avoid a second deprecation warning when
modifying the state of the app after it is started.
@odesenfans odesenfans changed the base branch from master to dev November 3, 2023 16:24
@odesenfans odesenfans changed the title Od enable werror CI: enable -Werror for pytest Nov 3, 2023
Problem: the internal AsyncIPFS client must be closed to avoid
triggering a warning.
Fixes a deprecation warning.
Fixes a SQLAlchemy deprecation warning.
Replace usage in chains/ethereum.py by `importlib.resources.load_text`.
ResourceWarning warnings occur in some tests, especially those related
to storage that rely on the aiohttp pytest client. The reason could not
be determined.
@odesenfans odesenfans marked this pull request as ready for review November 5, 2023 14:10
@odesenfans odesenfans merged commit 9aa8885 into dev Nov 5, 2023
2 checks passed
@odesenfans odesenfans deleted the od-enable-werror branch November 5, 2023 14:10
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.

1 participant