NotificationsYou must be signed in to change notification settings
Fork175
Star1.4k

uefi/test-runner: Remove`crate::` and`uefi::` prefix from Status#1714

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

nicholasbishop merged 1 commit intomainfrombishop-status-cleanup

Jun 29, 2025

Merged

uefi/test-runner: Remove`crate::` and`uefi::` prefix from Status#1714

nicholasbishop merged 1 commit intomainfrombishop-status-cleanup

Jun 29, 2025

Conversation

Copy link

Member

nicholasbishop commentedJun 28, 2025

In many places, especially doc comments,Status was referred to asuefi::Status orcrate::Status. Consistently useStatus instead, adding imports as needed. Note that in cases whereStatus is only used in docstrings, theuse is gated by#[cfg(doc)]. (This avoids needing a bunch of manualStatus::* links in docstrings.)

This is mostly a mechanical find-replace change, except for adding imports where needed.

Parts of this change are also present in#1603 and#1705.

Checklist

Sensible git history (for example, squash "typo" or "fix" commits). See theRewriting History guide for help.
Update the changelog (if necessary)

uefi/test-runner: Remove crate:: and uefi:: prefix from Status

c3df136

In many places, especially doc comments, `Status` was referred to as`uefi::Status` or `crate::Status`. Consistently use `Status` instead, addingimports as needed. Note that in cases where `Status` is only used in docstrings,the `use` is gated by `#[cfg(doc)]`. (This avoids needing a bunch of manual`Status::*` links in docstrings.)This is mostly a mechanical find-replace change, except for adding imports whereneeded.