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

Remove executable bit from files that definitely aren't #7756

Merged
merged 1 commit into from
Oct 13, 2017

Conversation

foolip
Copy link
Member

@foolip foolip commented Oct 13, 2017

That is:
*.css *.cur *.headers *.htm *.html *.js *.otf *.png *.woff *.xht *.xml

As a safety check, all touched files were checked for "#!/", and the
one matching file was left untouched:
css/work-in-progress/hixie/parsing-uri-index.png (actually a perl script)

That is:
*.css *.cur *.headers *.htm *.html *.js *.otf *.png *.woff *.xht *.xml

As a safety check, all touched files were checked for "#!/", and the
one matching file was left untouched:
css/work-in-progress/hixie/parsing-uri-index.png (actually a perl script)
@w3c-bots
Copy link

w3c-bots commented Oct 13, 2017

Build ERRORED

Started: 2017-10-13 10:16:37
Finished: 2017-10-13 10:59:28

Failing Jobs

  • firefox:nightly
  • chrome:unstable
  • safari:10.0
  • MicrosoftEdge:14.14393

View more information about this build on:

@foolip
Copy link
Member Author

foolip commented Oct 13, 2017

Travis is failing because of #7660, will ignore and merge manually.

@foolip foolip requested a review from jgraham October 13, 2017 11:21
@gsnedders gsnedders merged commit 0420d43 into master Oct 13, 2017
@gsnedders gsnedders deleted the executable-files branch October 13, 2017 12:01
jakearchibald pushed a commit to jakearchibald/web-platform-tests that referenced this pull request Nov 16, 2017
…-tests#7756)

That is:
*.css *.cur *.headers *.htm *.html *.js *.otf *.png *.woff *.xht *.xml

As a safety check, all touched files were checked for "#!/", and the
one matching file was left untouched:
css/work-in-progress/hixie/parsing-uri-index.png (actually a perl script)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants