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
style: Avoid hashing hashes. #18011
style: Avoid hashing hashes. #18011
Conversation
Heads up! This PR modifies the following files:
|
r? @SimonSapin or @nox |
@bors-servo r=nox |
📌 Commit 834db80 has been approved by |
☔ The latest upstream changes (presumably #17999) made this pull request unmergeable. Please resolve the merge conflicts. |
🔒 Merge conflict |
@bors-servo r=nox p=1 |
💡 This pull request was already approved, no need to approve it again.
|
📌 Commit 834db80 has been approved by |
@bors-servo r-
|
@bors-servo r=nox |
📌 Commit 9fcaa0d has been approved by |
style: Avoid hashing hashes. These are the last instances I can find of us hashing the atoms hash. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/18011) <!-- Reviewable:end -->
💔 Test failed - linux-rel-wpt |
@bors-servo r=nox
|
📌 Commit b90e51c has been approved by |
style: Avoid hashing hashes. These are the last instances I can find of us hashing the atoms hash. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/18011) <!-- Reviewable:end -->
💔 Test failed - linux-rel-css |
These are the last instances I can find of us hashing the atoms hash.
Sigh, now for real... @bors-servo r=nox |
📌 Commit de58c08 has been approved by |
style: Avoid hashing hashes. These are the last instances I can find of us hashing the atoms hash. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/18011) <!-- Reviewable:end -->
☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css1, mac-rel-css2, mac-rel-wpt1, mac-rel-wpt2, mac-rel-wpt3, mac-rel-wpt4, windows-msvc-dev |
These are the last instances I can find of us hashing the atoms hash.
This change is