Skip to content

[REBASE & FF] Add Patina Config Pursuant to RFC 25#80

Merged
os-d merged 3 commits intoOpenDevicePartnership:mainfrom
os-d:config
Feb 24, 2026
Merged

[REBASE & FF] Add Patina Config Pursuant to RFC 25#80
os-d merged 3 commits intoOpenDevicePartnership:mainfrom
os-d:config

Conversation

@os-d
Copy link
Contributor

@os-d os-d commented Feb 24, 2026

This PR implements the config portion of Patina RFC 25.

In preparation for this, it removes an unused section of config.toml and removes a not-globally-applicable part of the config. The git with cli configuration should be added to a developer's root config.toml if they desire it.

os-d added 3 commits February 24, 2026 08:44
The uefi-target-rules are currently unused, remove
them in preparation for adding config pursuant to
RFC 25.

Signed-off-by: Oliver Smith-Denny <osde@microsoft.com>
The git with cli config is useful for a subset of developers
who use credential managers. However, it is not globally
useful, so it is removed in preparation for adding config
pursuant to RFC 25.

Users of this config should add it to their root config.toml,
as this is a developer choice, not a project choice.

Signed-off-by: Oliver Smith-Denny <osde@microsoft.com>
This adds the global Patina configuration as described
in RFC 25.

Signed-off-by: Oliver Smith-Denny <osde@microsoft.com>
Copy link
Contributor

@vineelko vineelko left a comment

Choose a reason for hiding this comment

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

Once this is merged, may be we should remove the [profile.*] sections from cargo.toml files from all the repos.

@os-d
Copy link
Contributor Author

os-d commented Feb 24, 2026

Once this is merged, may be we should remove the [profile.*] sections from cargo.toml files from all the repos.

Yeah, I plan to intercept the file sync PR and add a commit to each, as in: OpenDevicePartnership/patina#1324.

@os-d os-d merged commit 711c1e6 into OpenDevicePartnership:main Feb 24, 2026
@os-d os-d deleted the config branch February 24, 2026 19:11
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.

4 participants