Problem/Motivation

For some reason, the files in this repo have the executable bit set. There's no reason for this, and this should be removed.

I executed the following command to generate the attached patch:

find . -type f | xargs chmod -x

Steps to reproduce

ls -l

Proposed resolution

find . -type f | xargs chmod -x
And then commit

CommentFileSizeAuthor
#4 after-patch.png14.34 KBdiegors
#4 before-patch.png152.54 KBdiegors
#2 admin_toolbar-remove-x-bit-3322693-2.patch5.13 KBjcnventura

Comments

jcnventura created an issue. See original summary.

jcnventura’s picture

jcnventura
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new admin_toolbar-remove-x-bit-3322693-2.patch5.13 KB
jcnventura’s picture

jcnventura
Primary language English
commented

Honestly, nothing to review here.. But would be nice if others gave it an RTBC.

diegors’s picture

diegors
He/Him
Primary language Portuguese, Brazil
Location Campinas - Brazil
commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new before-patch.png152.54 KB
new after-patch.png14.34 KB

Applied the patch and works.

Before patch:

After patch:

Moving to RTBC.

  • adriancid committed 39480ba8 on 3.x authored by jcnventura 
    Issue #3322693 by jcnventura, diegors, adriancid: Remove executable bit...
adriancid’s picture

adriancid
Primary language Spanish
Location Qc, Canada 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Thanks

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.