Skip to content
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

Support creation of containers and test containers on the global namespace #517

Merged

Conversation

lgrossi
Copy link
Contributor

@lgrossi lgrossi commented May 5, 2023

When creating a new build configuration without a namespace and adding later on a subnamespace to it (e.g. using getTestContainer() will add a subnamespace 'Test') the dumped file will have an invalid namespace '\Test' and that will make the di compiler fail.

image

@lcobucci
Copy link
Owner

lcobucci commented May 5, 2023

@lgrossi believe you pushed too many unnecessary files 😅

@lgrossi lgrossi force-pushed the lucas/fix-subnamespace-for-default-namespace branch 2 times, most recently from fecb467 to e5d19f0 Compare May 5, 2023 20:38
@lgrossi
Copy link
Contributor Author

lgrossi commented May 5, 2023

@lgrossi believe you pushed too many unnecessary files 😅

Ops, it was so simple that I did it without checking git diff. Totally forgot about .idea 😅.

Copy link
Owner

@lcobucci lcobucci left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a teeny tiny thing 👍

.gitignore Outdated Show resolved Hide resolved
@lcobucci lcobucci self-assigned this Aug 1, 2023
@lcobucci lcobucci added this to the 7.4.0 milestone Aug 1, 2023
@lcobucci lcobucci changed the title Fix subnamespace when there is no namespace to avoid issues with the base class Support creation of containers and test containers on the global namespace Aug 1, 2023
@lcobucci lcobucci force-pushed the lucas/fix-subnamespace-for-default-namespace branch 2 times, most recently from 179db62 to e335f06 Compare August 1, 2023 21:09
lgrossi and others added 2 commits August 1, 2023 23:20
…space

When creating a new build configuration without namespace and adding
later on a subnamespace to it (e.g. using getTestContainer() will add a
subnamespace 'Test') the dumped file will have an invalid namespace
'\Test' and that will make the di compiler fail.
Due to unexpected false-positives we will ignore the result

Signed-off-by: Luís Cobucci <lcobucci@gmail.com>
@lcobucci lcobucci force-pushed the lucas/fix-subnamespace-for-default-namespace branch from e335f06 to a833621 Compare August 1, 2023 21:21
@lcobucci lcobucci merged commit f5a4298 into lcobucci:7.4.x Aug 1, 2023
10 checks passed
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.

None yet

2 participants