Skip to content

Conversation

@llucax
Copy link
Contributor

@llucax llucax commented Apr 8, 2025

This repository doesn't use submodules anymore (in the past it had google common api as a submodule).

Copilot AI review requested due to automatic review settings April 8, 2025 12:03
@llucax llucax requested a review from a team as a code owner April 8, 2025 12:03
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

@github-actions github-actions bot added the part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) label Apr 8, 2025
@llucax llucax enabled auto-merge April 8, 2025 12:03
@llucax llucax added the type:tech-debt Improves the project without visible changes for users label Apr 8, 2025
@llucax llucax self-assigned this Apr 8, 2025
@llucax llucax requested a review from Marenz April 8, 2025 12:04
This repository doesn't use submodules anymore (in the past it had
google common api as a submodule).

Signed-off-by: Leandro Lucarella <[email protected]>
@Marenz Marenz force-pushed the rm-dependabot-submodule branch from 78ce645 to 2f19697 Compare April 9, 2025 12:27
Copy link
Contributor

@Marenz Marenz left a comment

Choose a reason for hiding this comment

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

LGTM. rebased

@llucax llucax added this pull request to the merge queue Apr 9, 2025
Merged via the queue into frequenz-floss:v0.x.x with commit dfce8fa Apr 9, 2025
6 checks passed
@llucax llucax deleted the rm-dependabot-submodule branch April 9, 2025 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) type:tech-debt Improves the project without visible changes for users

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants