Skip to content

Restructure (8): Make DB interface public #608

Merged
roosterfish merged 1 commit intocanonical:v3from
roosterfish:restructure_db_interface
Feb 4, 2026
Merged

Restructure (8): Make DB interface public #608
roosterfish merged 1 commit intocanonical:v3from
roosterfish:restructure_db_interface

Conversation

@roosterfish
Copy link
Contributor

@roosterfish roosterfish commented Feb 3, 2026

Can only be merged after (won't build before):

This is the eigth PR taken from a single commit in #534 (even though in this case the actual changes are very different from the original commit).
The final package layout will be different but to allow reviewing the changes in smaller chunks I will split this PR by its commits.

In this PR the DB interface is made public. It's returned by the public State interface and should therefore not be located inside the internal/ package.

Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
@roosterfish roosterfish force-pushed the restructure_db_interface branch from e37ed05 to c5e125e Compare February 3, 2026 16:45
@roosterfish roosterfish marked this pull request as ready for review February 3, 2026 16:46
@roosterfish roosterfish assigned markylaing and unassigned markylaing Feb 3, 2026
Copy link
Contributor

@tugbataluy tugbataluy left a comment

Choose a reason for hiding this comment

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

LGTM, thanks ❇️

@roosterfish roosterfish merged commit 2424a1b into canonical:v3 Feb 4, 2026
5 checks passed
@roosterfish roosterfish deleted the restructure_db_interface branch February 4, 2026 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants