Sitelet https://github.com/vivait/StringGeneratorBundle/pull/34
Skip to content

Fix AppBundle namespace issue - #34

Merged
Brunty merged 1 commit into
vivait:masterfrom
terox:fix-appbundle-namespace
Dec 11, 2017
Merged

Brunty merged 1 commit into
vivait:masterfrom
terox:fix-appbundle-namespace

Conversation

@terox

@terox terox commented Nov 30, 2017 •

Copy link
Copy Markdown
Contributor

Hello again,

Seems that 2.1 tag is broken. There are some issue with namespaces.

This are not detected by tests because they are not tested.

This will fix it.

Thank you

@terox

terox commented Nov 30, 2017

Copy link
Copy Markdown
Contributor Author

I am observing that are a little issues with CompilerPass also. I will try to investigate later. I think that we will need create another PR because it isn't related with that.

@terox

terox commented Dec 5, 2017

Copy link
Copy Markdown
Contributor Author

Ok my last message is related with old configuration, but nothing to worry.

This PR will fix the namespace issues.

Thank you

@Brunty

Brunty commented Dec 5, 2017

Copy link
Copy Markdown
Contributor

@leightonthomas can you double check this?

@terox

terox commented Dec 11, 2017

Copy link
Copy Markdown
Contributor Author

Hello,

Could be merged and tagged?

Thank you ;)

@Brunty
Brunty merged commit d2a501c into vivait:master Dec 11, 2017
@Brunty

Brunty commented Dec 11, 2017

Copy link
Copy Markdown
Contributor

@terox done, thanks! (apologies for the delay) :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants