Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Both #2056415: Remove Unused local variable $function from /core/update.php and #2056417: Remove Unused local variable $person_uri from /core/modules/rdf/lib/Drupal/rdf/Tests/UserAttributesTest.php introduced incorrect file permissions. I've developed a git pre-commit hook to get so I don't do this anymore. Attached for others to this issue.
Comment | File | Size | Author |
---|---|---|---|
#4 | 2057445.file-perms.4.patch | 614 bytes | alexpott |
#3 | 2057445.file-perms.3.patch | 351 bytes | alexpott |
#1 | 2057445.file-perms.1.patch | 248 bytes | alexpott |
pre-commit.txt | 686 bytes | alexpott |
Comments
Comment #1
alexpottPatch attached
Comment #2
ParisLiakos CreditAttribution: ParisLiakos commentedwe could include fix for #2039091: drupal_bootstrap docs are missing phases here as well
Comment #3
alexpottCertainly!
Comment #4
alexpottWell let's clean up the one remaining!!!
Ignore /vendor and /scripts we have
./core//modules/aggregator/tests/modules/aggregator_test_views/test_views/views.view.test_aggregator_items.yml
Comment #5
ParisLiakos CreditAttribution: ParisLiakos commentedawesomeness. thanks
Comment #6
webchickWow. Sorry about that.
Committed and pushed to 8.x. Thanks!