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 to repo-config v0.11.0 #1111

Merged
merged 2 commits into from
Nov 21, 2024
Merged

Conversation

llucax
Copy link
Contributor

@llucax llucax commented Nov 21, 2024

Update the files using the migration script with the following changes:

  • Group artifact-related actions so they are updated together.
  • Add asyncio_default_fixture_loop_scope pytest option as not specifying this is deprecated.
  • Disable some pylint checks that are already checked by mypy.

We also bump the build dependencies, so the distribution files use names that are compatible with PEP625, which will soon be required by PyPI and fix pylint warnings in examples that were previously ignored as they were not checked by pylint due to a sybil bug.

@llucax llucax requested a review from a team as a code owner November 21, 2024 12:34
@llucax llucax requested review from ela-kotulska-frequenz and removed request for a team November 21, 2024 12:34
@llucax llucax self-assigned this Nov 21, 2024
@github-actions github-actions bot added part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) part:actor Affects an actor ot the actors utilities (decorator, etc.) labels Nov 21, 2024
@llucax llucax added this to the v1.0.0-rc1301 milestone Nov 21, 2024
@llucax llucax enabled auto-merge November 21, 2024 12:34
@llucax llucax added the cmd:skip-release-notes It is not necessary to update release notes for this PR label Nov 21, 2024
@llucax
Copy link
Contributor Author

llucax commented Nov 21, 2024

Skipping release notes as there are no visible changes.

Also update the files using the migration script with the following
changes:

* Group artifact-related actions so they are updated together.
* Add `asyncio_default_fixture_loop_scope` `pytest` option as not
  specifying this is deprecated.
* Disable some `pylint` checks that are already checked by `mypy`.

We also bump the build dependencies so the distribution files use names
that are compatible with PEP625, which will soon be required by PyPI.

Signed-off-by: Leandro Lucarella <[email protected]>
The `__init__.py` files were not previously checked with `pylint`
because of sybil bug.

Signed-off-by: Leandro Lucarella <[email protected]>
@llucax llucax added this pull request to the merge queue Nov 21, 2024
Merged via the queue into frequenz-floss:v1.x.x with commit 0ad2ac7 Nov 21, 2024
18 checks passed
@llucax llucax deleted the repo-config-0.11 branch November 21, 2024 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cmd:skip-release-notes It is not necessary to update release notes for this PR part:actor Affects an actor ot the actors utilities (decorator, etc.) part:tooling Affects the development tooling (CI, deployment, dependency management, etc.)
Projects
Development

Successfully merging this pull request may close these issues.

2 participants