Skip to content

Bump the api-common dependency to v0.5.4#205

Merged
leandro-lucarella-frequenz merged 1 commit intofrequenz-floss:v0.15.xfrom
llucax:bump-common-v0.5
Feb 1, 2024
Merged

Bump the api-common dependency to v0.5.4#205
leandro-lucarella-frequenz merged 1 commit intofrequenz-floss:v0.15.xfrom
llucax:bump-common-v0.5

Conversation

@llucax
Copy link
Copy Markdown
Contributor

@llucax llucax commented Jan 30, 2024

The new v0.5.4 release of the api-common repository is now backwards compatible with the old v0.3.x releases. We bump the dependency so microgrid API can be used together with other APIs that depend on the newer common API version.

Please note that the googleapis-common-protos dependency is also bumped to v1.56.4, which is the version that api-common v0.5.4 depends on.

Strictly speaking, this is a breaking change too, as projects might need to bump their googleapis-common-protos dependency to v1.56.4 too if they specify it explicitly, but this is highly unlikely and very easy to fix.

@llucax llucax self-assigned this Jan 30, 2024
@github-actions github-actions Bot added part:docs Affects the documentation part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) labels Jan 30, 2024
@llucax llucax added this to the v0.15.3 milestone Jan 30, 2024
@llucax llucax changed the base branch from v0.x.x to v0.15.x January 30, 2024 11:26
@llucax

This comment was marked as outdated.

@llucax
Copy link
Copy Markdown
Contributor Author

llucax commented Jan 30, 2024

I guess it was because I made the PR originally against the wrong branch, now I force-pushed and they are running...

@llucax
Copy link
Copy Markdown
Contributor Author

llucax commented Jan 30, 2024

This needs updating the dependency when common API v0.5.4 is released. The test with nox CI check also fails because we need a release of common API v0.5.4 first for it to pass (otherwise it can't parse the minimum version from the string).

The tests that are not running are just because we are in an old branch that used different check names.

@llucax llucax changed the title Bump the api-common dependency to v0.5.3 Bump the api-common dependency to v0.5.4 Jan 31, 2024
@llucax llucax force-pushed the bump-common-v0.5 branch 2 times, most recently from be5886c to 5cce4b3 Compare January 31, 2024 15:17
The new v0.5.4 release of the api-common repository is now backwards
compatible with the old v0.3.x releases. We bump the dependency so
microgrid API can be used together with other APIs that depend on the
newer common API version.

Please note that the googleapis-common-protos dependency is also bumped
to v1.56.4, which is the version that api-common v0.5.4 depends on.

Strictly speaking, this is a breaking change too, as projects might need
to bump their googleapis-common-protos dependency to v1.56.4 too if they
specify it explicitly, but this is highly unlikely and very easy to fix.

Signed-off-by: Leandro Lucarella <luca-frequenz@llucax.com>
@llucax llucax marked this pull request as ready for review January 31, 2024 15:20
@llucax llucax requested review from a team as code owners January 31, 2024 15:20
@llucax llucax enabled auto-merge January 31, 2024 15:20
@llucax llucax disabled auto-merge January 31, 2024 15:21
@llucax
Copy link
Copy Markdown
Contributor Author

llucax commented Jan 31, 2024

API common v0.5.4 was released, so this PR is updated too and ready for review (and merge).

@llucax llucax added the priority:high Address this as soon as possible label Jan 31, 2024
@llucax
Copy link
Copy Markdown
Contributor Author

llucax commented Jan 31, 2024

FYI, there are a couple of CI checks that are expected but will never run because they are not present in this branch, but all tests are passing. We'll need to force-merge once this is approved.

@llucax
Copy link
Copy Markdown
Contributor Author

llucax commented Feb 1, 2024

I will force-merge this, as the tests will never complete.

@leandro-lucarella-frequenz leandro-lucarella-frequenz merged commit 2332734 into frequenz-floss:v0.15.x Feb 1, 2024
@leandro-lucarella-frequenz leandro-lucarella-frequenz deleted the bump-common-v0.5 branch February 1, 2024 09:59
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.) priority:high Address this as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants