Skip to content

Fix xml blocks #7915

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

Merged
merged 1 commit into from
May 18, 2017
Merged

Fix xml blocks #7915

merged 1 commit into from
May 18, 2017

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