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

use single quotes for YAML strings #5955

Closed
wants to merge 2 commits into from
Closed

use single quotes for YAML strings #5955

wants to merge 2 commits into from

Conversation

snoek09
Copy link

@snoek09 snoek09 commented Dec 5, 2015

Q A
Doc fix? yes
New docs? no
Applies to all
Fixed tickets #5935

@wouterj
Copy link
Member

wouterj commented Dec 5, 2015

👍 (there are a lot more double quote usages in the docs though, if you have more time and did enjoy creating this PR, feel free to fix them aswell! 😉)

xabbuh added a commit that referenced this pull request Dec 9, 2015
This PR was squashed before being merged into the 2.3 branch (closes #5955).

Discussion
----------

use single quotes for YAML strings

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | all
| Fixed tickets | #5935

Commits
-------

81b96b6 use single quotes for YAML strings
@xabbuh
Copy link
Member

xabbuh commented Dec 9, 2015

Thank you very much for doing this boring task @snoek09! I have squashed your commits while merging so that's why your pull request is shown as closed. If you like to, we will be happy to merge other pull requests by you which update more quotes as well. :)

@xabbuh xabbuh closed this Dec 9, 2015
@snoek09 snoek09 deleted the single-quotes-yaml-strings branch December 9, 2015 08:01
leofeyer added a commit to contao/contao that referenced this pull request Feb 19, 2019
leofeyer added a commit to contao/calendar-bundle that referenced this pull request Feb 19, 2019
leofeyer added a commit to contao/news-bundle that referenced this pull request Feb 19, 2019
leofeyer added a commit to contao/installation-bundle that referenced this pull request Feb 19, 2019
leofeyer added a commit to contao/manager-bundle that referenced this pull request Feb 19, 2019
leofeyer added a commit to contao/faq-bundle that referenced this pull request Feb 19, 2019
leofeyer added a commit to contao/core-bundle that referenced this pull request Feb 19, 2019
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.

3 participants