Skip to content

Conversation

@flora-hofmann-frequenz
Copy link
Collaborator

No description provided.

@github-actions github-actions bot added part:docs Affects the documentation part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) labels Nov 28, 2024
Copy link
Contributor

@cwasicki cwasicki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't there anything else to update? IIUC there is a migration script

pyproject.toml Outdated
"setuptools_scm[toml] == 7.1.0",
"setuptools == 75.5.0",
"setuptools_scm[toml] == 8.1.0",
"frequenz-repo-config[lib] == 0.10.0",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what about this version?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doesn't this need updating?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is updating according to the repo config specs. 75.5.0 and 8.1.0 are the versions required.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is also in PR #119 and #120 from Dependabot.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mean why is repo config not updated here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I thought you meant the setup tools update... Will check!

@flora-hofmann-frequenz
Copy link
Collaborator Author

Isn't there anything else to update? IIUC there is a migration script

I've ran the migration script. Could you please review again?

@flora-hofmann-frequenz flora-hofmann-frequenz added this pull request to the merge queue Dec 2, 2024
Merged via the queue into frequenz-floss:v0.x.x with commit caddfa8 Dec 2, 2024
14 checks passed
@flora-hofmann-frequenz flora-hofmann-frequenz deleted the update_repo branch December 2, 2024 09:39
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