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.

Sign up for GitHub

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

Jump to bottom

test: standardize to 8 character commit SHA1 in assert_contributor_names #314

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

test: standardize to 8 character commit SHA1 in assert_contributor_names #314

wants to merge 3 commits into from
+1,508 −1,412

Commits on Oct 8, 2024

  1. test: standardize to 8 character commit SHA1 in assert_contributor_names 

    ⚗️ Proof of concept SHA1 lengthening script

♻️ Decouple from specific test file

run whole suite instrumenting tests which include AssertContributorNames

🚚 Move to bin/lengthen-sha1, drop thor dependency

🚨 Fix whitespace

✏️ Missed dropping thor/action
    @cpb
    cpb committed Oct 8, 2024
    Configuration menu
    Copy the full SHA
    90b7d78 View commit details
    Browse the repository at this point in the history

  2. update calls to assert_contributor_names to have 8 character SHA1 

    using bin/lengthen-sha1
    @cpb
    cpb committed Oct 8, 2024
    Configuration menu
    Copy the full SHA
    b45deaa View commit details
    Browse the repository at this point in the history

  3. typo: missed refactoring to target_length

    @cpb
    cpb committed Oct 8, 2024
    Configuration menu
    Copy the full SHA
    c39e947 View commit details
    Browse the repository at this point in the history