8000 Fix xml blocks by GuilhemN · Pull Request #7915 · symfony/symfony-docs · GitHub
[go: up one dir, main page]

8000
Skip to content

Conversation

@GuilhemN
Copy link
Contributor

No description provided.

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.

@GuilhemN wow!! You must have spent a lot of time fixing all this. Thanks a lot! 🙇

And the mandatory comment when dealing with XML: I don't know how people can use this config format. It's beyond verbose ... it's ridiculously absurd.

@xabbuh
Copy link
Member
xabbuh commented May 18, 2017

Wow, thank you so much Guilhem!

@xabbuh xabbuh merged commit 2bf7529 into symfony:2.7 May 18, 2017
xabbuh added a commit that referenced this pull request May 18, 2017
This PR was merged into the 2.7 branch.

Discussion
----------

Fix xml blocks

Commits
-------

2bf7529 Fix xml blocks
@GuilhemN
Copy link
Contributor Author
GuilhemN commented May 18, 2017

wow!! You must have spent a lot of time fixing all this.

A bit 👼 but a proper search helped a lot :)

And the mandatory comment when dealing with XML: I don't know how people can use this config format. It's beyond verbose ... it's ridiculously absurd.

Completely agree... if I understood well, some IDEs have nice features motivating people to use it.

@GuilhemN GuilhemN deleted the FIXXML2 branch May 18, 2017 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@javiereguiluz javiereguiluz javiereguiluz approved these changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

0