Skip to content

Conversation

@bocharsky-bw
Copy link
Member

Let's see if tests pass, but most probably will need to fix some things to make them green with this change

@bocharsky-bw
Copy link
Member Author

Easy win, tests passed now! A lot of good changes were merged lately

@welcoMattic
Copy link
Member

Well done @bocharsky-bw!

@welcoMattic welcoMattic merged commit 305e31f into php-translation:master Jul 26, 2019
@XWB
Copy link
Contributor

XWB commented Jul 26, 2019

The issue is still not resolved for people with Twig though (like almost everyone). Removing dependencies is not the solution.

@bocharsky-bw bocharsky-bw deleted the allow-any-twig branch July 26, 2019 16:25
@bocharsky-bw
Copy link
Member Author

@XWB wait, what Twig issue are you talking about? Actually, it wasn't a fix, it was just a partial revert of changes I've done before to make tests green. See #313 (comment)

@bocharsky-bw
Copy link
Member Author

And while tests are green now we can focus on specific problems and improve this bundle step by step. And this is one of such steps

@Nyholm Nyholm mentioned this pull request Sep 5, 2019
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