Skip to content

Fix line ending style #203

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.

Sign up for GitHub

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
Merged
merged 1 commit into from
Mar 10, 2021
Merged

Fix line ending style #203

merged 1 commit into from
Mar 10, 2021

Conversation

@renatho
Copy link
Contributor

@renatho renatho commented Mar 10, 2021

Changes proposed in this Pull Request

  • It just updates line ending in some files. It was causing some issues to add the file to SVN.

Testing instructions

  • Go to wp-admin > Settings > SyntaxHighlighter.
  • Update the option "Highlighter Version" to "Version 2.x".
  • Test if the code continue formatted in the frontend.

@renatho renatho added the [Type] Maintenance Changes to non-production files label Mar 10, 2021
@renatho renatho added this to the v3.6.0 milestone Mar 10, 2021
@renatho renatho self-assigned this Mar 10, 2021
alexsanford
alexsanford approved these changes Mar 10, 2021
View reviewed changes
Copy link
Contributor

@alexsanford alexsanford left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems safe 👍

@renatho renatho merged commit 6e370c0 into master Mar 10, 2021
@renatho renatho deleted the fix/line-ending-style branch March 10, 2021 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gikaragia gikaragia Awaiting requested review from gikaragia

1 more reviewer

@alexsanford alexsanford alexsanford approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@renatho renatho

Labels

[Type] Maintenance Changes to non-production files

Projects

None yet

Milestone

v3.6.0

Development

Successfully merging this pull request may close these issues.

3 participants

@renatho @alexsanford