Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Clean up #25652

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

Closed
wants to merge 2 commits into from
Closed

Clean up #25652

wants to merge 2 commits into from

Conversation

carusogabriel
Copy link
Contributor

Q A
Branch? 2.7
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT
Doc PR -

I've removed all white spaces and extra lines from our markdown files. Recently I did it in our documentation, and thought would be interesting to replicate here 😄

Copy link
Member

@javiereguiluz javiereguiluz left a comment

Choose a reason for hiding this comment

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

I like this ... but the changes to the Markdown files stored in Tests/ dir should be reverted because otherwise tests will fail (these Markdown are the results or running some command, so we cannot modify them in any way or the command output won't match). Thanks.

@carusogabriel
Copy link
Contributor Author

@javiereguiluz Sorry for touch them, I've remove those changes 😅

@nicolas-grekas nicolas-grekas added this to the 2.7 milestone Jan 2, 2018
@nicolas-grekas
Copy link
Member

Thank you @carusogabriel.

nicolas-grekas added a commit that referenced this pull request Jan 2, 2018
This PR was squashed before being merged into the 2.7 branch (closes #25652).

Discussion
----------

Clean up

| Q             | A
| ------------- | ---
| Branch?       | 2.7
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

I've removed all white spaces and extra lines from our `markdown` files. Recently [I did it in our documentation](symfony/symfony-docs#8944), and thought would be interesting to replicate here 😄

Commits
-------

acd0d7c Clean up
@carusogabriel carusogabriel deleted the clean-up branch January 2, 2018 17:38
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