Skip to content

SPR-11627 fix #630

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

Closed
wants to merge 2 commits into from
Closed

SPR-11627 fix #630

wants to merge 2 commits into from

Conversation

asarkar
Copy link

@asarkar asarkar commented Apr 3, 2014

GroovyDynamicElementReader tries to read a Groovy script as XML and fails - SPR-11627

@asarkar asarkar mentioned this pull request Apr 3, 2014
@dsyer
Copy link
Member

dsyer commented Apr 4, 2014

Did you fill in the contributor's agreement yet?

@philwebb philwebb added this to the 1.0.1 milestone Apr 4, 2014
@asarkar
Copy link
Author

asarkar commented Apr 4, 2014

I didn't because the README said that it's not required for minor changes. Since you asked, I'm going to do it anyway.
I see couple tests failed. I ran the spring-boot build before pushing and it worked. This seems to be the integration tests which I didn't run locally. I'll check what's going on later today.

@dsyer
Copy link
Member

dsyer commented Apr 4, 2014

Thanks. The README actually says "please fill out the CA before your first change however trivial". If that's not clear enough we should change it.

@dsyer dsyer closed this in 430fb51 Apr 4, 2014
@asarkar
Copy link
Author

asarkar commented Apr 4, 2014

My bad, I misunderstood it. The language is perfectly fine. I submitted the ICA, confirmation # 77520140404030015.
Is this pull closed because it has been merged or it wasn't accepted?

@philwebb
Copy link
Member

philwebb commented Apr 4, 2014

Cheers. It was merged :)

430fb51

gigfork pushed a commit to boostrack/spring-boot that referenced this pull request Apr 21, 2014
Since it is not parsing XML it makes no sense to be validating
and it would cause exceptions when importing "namespace"
features. This brings the Boot usage of the bean builder inline
with that in Spring Core.

Fixes spring-projectsgh-630
mdeinum pushed a commit to mdeinum/spring-boot that referenced this pull request Jun 6, 2014
Since it is not parsing XML it makes no sense to be validating
and it would cause exceptions when importing "namespace"
features. This brings the Boot usage of the bean builder inline
with that in Spring Core.

Fixes spring-projectsgh-630
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants