Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Formatting fixes #741

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Formatting fixes #741

wants to merge 4 commits into from

Conversation

mkopec
Copy link
Member

@mkopec mkopec commented Jan 17, 2024

Sort tests alphabetically for easier browsing (previously they were roughly sorted by ID), and replace Coreboot with coreboot.

Signed-off-by: Michał Kopeć <[email protected]>
@mkopec mkopec changed the title Sort tests alphabetically Formatting fixes Jan 17, 2024
@mkopec
Copy link
Member Author

mkopec commented Jan 17, 2024

Also decided to replace Nvidia and NVidia with NVIDIA as per their guidelines

@macpijan
Copy link
Contributor

@PLangowski

Also decided to replace Nvidia and NVidia with NVIDIA as per their guidelines

Another one to add to your hooks :))

@macpijan
Copy link
Contributor

macpijan commented Jun 17, 2024

@BeataZdunczyk This is already old, and needs a rebase. It may also get in conflict with #828 which also changes lots of stuff in test documentation. We should make sure to cleanup these PRs more often, and ensure we do not trigger conflicts with older PRs, to not waste resources.

I am afraid of merging this right now, so it does not break what @JanPrusinowski has published in #828

How about @JanPrusinowski will integrate both of these changes, that can be also a fine git exercise 😃

@macpijan
Copy link
Contributor

macpijan commented Jun 17, 2024

Coreboot with coreboot.

Things like that shall be covered by (detected and fixed) hook from @PLangowski automatically in the future, if we enable this in pre-commit in this repo @BeataZdunczyk CC

https://github.com/3mdeb/pre-commit-hook-branding/tree/namespell/namespell

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