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

Jump to bottom

[wptrunner] Replace CRLF with LF in a few files #27190

Merged
merged 1 commit into from Jan 14, 2021
Merged

[wptrunner] Replace CRLF with LF in a few files #27190

merged 1 commit into from Jan 14, 2021

Conversation

stephenmcgruer
Copy link
Contributor

These aren't harmful, but we're mostly consistently using LF and this
annoys my editor :D

PR generated via find tools/wptrunner -type f | xargs sed -i 's/\r$//g'

@stephenmcgruer
[wptrunner] Replace CRLF with LF in a few files
011a72b 
These aren't harmful, but we're mostly consistently using LF and this
annoys my editor :D
@wpt-pr-bot wpt-pr-bot added infra wptrunner The automated test runner, commonly called through ./wpt run labels Jan 14, 2021
@stephenmcgruer stephenmcgruer merged commit f333959 into master Jan 14, 2021
@stephenmcgruer stephenmcgruer deleted the smcgruer/fix-crlf branch January 14, 2021 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LukeZielinski LukeZielinski LukeZielinski approved these changes

@jgraham jgraham Awaiting requested review from jgraham

Assignees

@LukeZielinski LukeZielinski

Labels
infra wptrunner The automated test runner, commonly called through ./wpt run
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@stephenmcgruer @LukeZielinski @wpt-pr-bot