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

Merge v0.17.1 into v0.x.x #256

Merged
merged 7 commits into from
Sep 11, 2024
Merged

Conversation

llucax
Copy link
Contributor

@llucax llucax commented Sep 10, 2024

  • Clear release notes
  • Bump frequenz-api-common from 0.6.1 to 0.6.2
  • Reformat pyproject.toml using taplo
  • Pin grpc related dependencies
  • Bump mkdocstrings dependencies

llucax and others added 6 commits September 10, 2024 14:41
Signed-off-by: Leandro Lucarella <[email protected]>
This is to make sure we pull a fix for the `protobuf` and `grpcio`
versions used to generate the python files.

Signed-off-by: Leandro Lucarella <[email protected]>
Signed-off-by: Leandro Lucarella <[email protected]>
We can't use wide dependencies because otherwise when building the
wheel, the latest version is used, but the generate code should use the
minimum supported version for the generation, not the latest one. See:
https://protobuf.dev/support/cross-version-runtime-guarantee/

Signed-off-by: Leandro Lucarella <[email protected]>
This is to avoid an issue with some outdated `griffe` imports.

While upgrading the dependency we need to remove the `custom_templates`
option that we don't use, as it was previously ignored but now the
plugin checks if the directory exists and fails if it doesn't.

Signed-off-by: Leandro Lucarella <[email protected]>
@llucax llucax requested review from a team as code owners September 10, 2024 13:16
@llucax llucax requested review from thea-leake and daniel-zullo-frequenz and removed request for a team September 10, 2024 13:16
* tag 'v0.17.1':
  Bump mkdocstrings dependencies
  Pin grpc related dependencies
  Reformat `pyproject.toml` using taplo
  Bump frequenz-api-common from 0.6.1 to 0.6.2
  Clear release notes

Signed-off-by: Leandro Lucarella <[email protected]>
@github-actions github-actions bot added part:docs Affects the documentation part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) labels Sep 10, 2024
@llucax llucax self-assigned this Sep 10, 2024
@llucax llucax enabled auto-merge September 10, 2024 13:16
@tiyash-basu-frequenz
Copy link
Contributor

Why was the v0.17.x branch created? It could all have been done on v0.x.x.

@llucax
Copy link
Contributor Author

llucax commented Sep 11, 2024

Why was the v0.17.x branch created? It could all have been done on v0.x.x.

As said on Slack only as an oversight and following the default behavior that v0.x.x is the development branch for the next minor, not the next patch version. I didn't check the commit history for v0.x.x so I didn't know it only had some devel dependency updates on top of v0.17.0.

@tiyash-basu-frequenz
Copy link
Contributor

As mentioned on slack, we can take this in, and immediately follw with breaking changes.

@llucax llucax added this pull request to the merge queue Sep 11, 2024
@llucax llucax added this to the v0.18.0 milestone Sep 11, 2024
Merged via the queue into frequenz-floss:v0.x.x with commit 6291e8e Sep 11, 2024
12 checks passed
@llucax llucax deleted the merge-0.17.1 branch September 11, 2024 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
part:docs Affects the documentation part:tooling Affects the development tooling (CI, deployment, dependency management, etc.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants