Skip to content

Conversation

@kristofvc
Copy link
Contributor

Q A
Doc fix? yes
New docs? no
Applies to 2.3
Fixed tickets #4251
Copy link
Member

Choose a reason for hiding this comment

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

the ``International Components for Unicode_ should be put on the next line (in order to not exceed our 72th characters line rule)

@kristofvc
Copy link
Contributor Author

I put the second part of the sentence on a new line if that's ok

@wouterj
Copy link
Member

wouterj commented Oct 2, 2014

Looks good

@xabbuh
Copy link
Member

xabbuh commented Oct 2, 2014

👍 I like it.

@weaverryan
Copy link
Member

Great work Kristof - thanks for fixing this!

@weaverryan weaverryan merged commit db85c67 into symfony:2.3 Oct 3, 2014
weaverryan added a commit that referenced this pull request Oct 3, 2014
…rgh, kristofvc) This PR was merged into the 2.3 branch. Discussion ---------- change misleading language identifier | Q | A | ------------- | --- | Doc fix? | yes | New docs? | no | Applies to | 2.3 | Fixed tickets | #4251 Commits ------- db85c67 formatting to follow 72 characters line limit 12cccc8 changed dash to underscore d68cd09 change misleading language identifier
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

4 participants