Skip to content

Conversation

@NotTheEvilOne
Copy link
Contributor

@NotTheEvilOne NotTheEvilOne commented Apr 28, 2025

What this PR does / why we need it:
This PR restructures python_gardenlinux_lib from the ground up. It:

  • Moves the python_gardenlinux_lib package to gardenlinux as it is a Python package anyway and does not only contain or use library functionality.
  • Introduces Python classes for GL features and flavors parsing.

@NotTheEvilOne NotTheEvilOne self-assigned this Apr 28, 2025
@NotTheEvilOne NotTheEvilOne force-pushed the feature/gardenlinux-restructure branch from 14b2355 to c9a8c8f Compare April 28, 2025 14:53
@NotTheEvilOne NotTheEvilOne force-pushed the feature/gardenlinux-restructure branch from c9a8c8f to 7dbe148 Compare April 28, 2025 14:58
Copy link
Contributor

@yeoldegrove yeoldegrove left a comment

Choose a reason for hiding this comment

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

I like the restructuring approach.
In general I would not take too much efforts to keep this backward compatible.
The few places where this is used can be updated easily at the moment.
Also, we have tagged releases https://github.com/gardenlinux/python-gardenlinux-lib/tags that can be consumed by anything that uses this.

@NotTheEvilOne NotTheEvilOne force-pushed the feature/gardenlinux-restructure branch from 7dbe148 to 58e2994 Compare May 6, 2025 06:28
@NotTheEvilOne NotTheEvilOne force-pushed the feature/gardenlinux-restructure branch from 58e2994 to b41949c Compare May 6, 2025 07:07
Signed-off-by: Tobias Wolf <[email protected]>
@NotTheEvilOne NotTheEvilOne force-pushed the feature/gardenlinux-restructure branch 3 times, most recently from eed9dd4 to 0a8bf98 Compare May 15, 2025 13:54
@NotTheEvilOne NotTheEvilOne force-pushed the feature/gardenlinux-restructure branch from 0a8bf98 to 78a7ae2 Compare May 15, 2025 13:56
@NotTheEvilOne NotTheEvilOne marked this pull request as ready for review May 15, 2025 13:59
@NotTheEvilOne NotTheEvilOne merged commit 5fc692d into main May 15, 2025
9 checks passed
@NotTheEvilOne NotTheEvilOne deleted the feature/gardenlinux-restructure branch May 15, 2025 14:08
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