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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants