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

Fix theming autoloader magic #1033

Merged
merged 2 commits into from
Aug 24, 2016
Merged

Fix theming autoloader magic #1033

merged 2 commits into from
Aug 24, 2016

Conversation

rullzer
Copy link
Member

@rullzer rullzer commented Aug 24, 2016

Fixes #1032

Basically to agressive composer caching kills us here.

CC: @juliushaertl @blizzz @nickvergessen @LukasReschke

This also needs to go into stable10 else theming is basically broken.

@blizzz
Copy link
Member

blizzz commented Aug 24, 2016

a bit magical, but works and best we can do short term 👍 thanks @rullzer

@blizzz
Copy link
Member

blizzz commented Aug 24, 2016

@juliushaertl can you verify?

@juliushaertl
Copy link
Member

yes, works fine here 👍

blizzz pushed a commit that referenced this pull request Aug 24, 2016
Dark autoloader magic for ThemingDefaults

Require a ThemingDefaults class again
@blizzz
Copy link
Member

blizzz commented Aug 24, 2016

Backport is here: #1034

@LukasReschke
Copy link
Member

🙈 👍

@rullzer rullzer added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Aug 24, 2016
@rullzer rullzer merged commit 42ef336 into master Aug 24, 2016
@rullzer rullzer deleted the fix_theming_autoloader_magic branch August 24, 2016 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants