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.

Sign up for GitHub

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

Jump to bottom

Remove unused google api-common-protos submodule #58

Closed
llucax opened this issue Oct 7, 2024 · 0 comments · Fixed by #61
Closed

Remove unused google api-common-protos submodule #58

llucax opened this issue Oct 7, 2024 · 0 comments · Fixed by #61
Labels
type:tech-debt Improves the project without visible changes for users
Milestone
v1.0.0

Comments

@llucax
Copy link
Contributor

llucax commented Oct 7, 2024

This submodule is not needed anymore. The python dependency was already removed.
@llucax llucax added the type:tech-debt Improves the project without visible changes for users label Oct 7, 2024
@cwasicki cwasicki added this to the v1.0.0 milestone Oct 11, 2024
stefan-brus-frequenz added a commit to stefan-brus-frequenz/frequenz-api-reporting that referenced this issue Nov 21, 2024
@stefan-brus-frequenz
Remove unused api-common-protos submodule
27100fa 
Fixes frequenz-floss#58
Merged
stefan-brus-frequenz added a commit to stefan-brus-frequenz/frequenz-api-reporting that referenced this issue Nov 21, 2024
@stefan-brus-frequenz
Remove unused api-common-protos submodule
a9f4cbb 
Fixes frequenz-floss#58

Signed-off-by: Stefan Brus <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type:tech-debt Improves the project without visible changes for users
Projects
None yet
Milestone
v1.0.0
Development

Successfully merging a pull request may close this issue.

Remove unused api-common-protos submodule stefan-brus-frequenz/frequenz-api-reporting
2 participants
@llucax @cwasicki