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

coverage: Format all coverage tests with rustfmt #120015

Merged
merged 3 commits into from
Jan 21, 2024
Merged

Commits on Jan 16, 2024

  1. coverage: Add #[rustfmt::skip] to tests with non-standard formatting

    These tests deliberately use non-standard formatting, so that the line
    execution counts reported by `llvm-cov` reveal additional information about
    where code regions begin and end.
    Zalathar committed Jan 16, 2024
    Configuration menu
    Copy the full SHA
    f149442 View commit details
    Browse the repository at this point in the history
  2. coverage: Tweak individual tests to be unaffected by rustfmt

    Some of these tests use non-standard formatting that we can simulate by
    strategically adding `//` line comments.
    
    One contains `where` clauses that would be split across multiple lines, which
    we can keep on one line by moving the bounds to the generic type instead.
    Zalathar committed Jan 16, 2024
    Configuration menu
    Copy the full SHA
    1f9353a View commit details
    Browse the repository at this point in the history

Commits on Jan 17, 2024

  1. coverage: Format all remaining tests

    These tests can simply be reformatted as normal, because the resulting changes
    are unimportant.
    Zalathar committed Jan 17, 2024
    Configuration menu
    Copy the full SHA
    99797bb View commit details
    Browse the repository at this point in the history