Skip to content

Conversation

@avitacco
Copy link
Contributor

@avitacco avitacco commented Oct 9, 2025

This PR makes many changes that are outside of the scope of the issue mentioned above. They are improvements, but if you disagree, please feel free to cherry-pick.

  • [Change] Switched to version 11 of the apt module, as it is the first to support the new key requirements
  • [Change] Changed the apt::source resource in repo.pp to install the key in such a way as to avoid the deprecation notice from apt
  • [Change] Updated PDK to v3.4.0 because it is the most recent non-closed-source version of PDK
  • [Addition] Added missing documentation for parameters
  • [Change] Changed the class/parameter documentation to match modern code standards
  • [Change] Removed anchor resources and replaced them with contains because anchors are deprecated and have been for years
  • [Change] Updated tests to match the anchor to contain changes
  • [Change] A lot of small code changes to satisfy pdk validate
  • [Change] Increased the max version for the archive module

This PR makes many changes that are outside of the scope of the
issue mentioned above. They are improvements, but if you disagree, please feel free to cherry-pick.

* [Change] Switched to version 11 of the apt module, as it is the first
  to support the new key requirements
* [Change] Changed the apt::source resource in repo.pp to install the
  key in such a way as to avoid the deprecation notice from apt
* [Change] Updated PDK to v3.4.0 because it is the most recent
  non-closed-source version of PDK
* [Addition] Added missing documentation for parameters
* [Change] Changed the class/parameter documentation to match modern
  code standards
* [Change] Removed anchor resources and replaced them with contains because anchors are deprecated and have been for years
* [Change] Updated tests to match the anchor to contain changes
* [Change] A lot of small code changes to satisfy pdk validate
* [Change] Increased the max version for the archive module
@pcfens pcfens merged commit 28a7007 into pcfens:master Oct 9, 2025
2 checks passed
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.

2 participants