Skip to content

Conversation

@Guite
Copy link
Contributor

@Guite Guite commented Jan 22, 2020

This PR adds a service definition for the extractor wanted in #394

Do not merge before php-translation/extractor#148 is merged.

@odolbeau
Copy link
Member

Version 2.0.1 of extractor released!
https://github.com/php-translation/extractor/releases/tag/2.0.1

@Guite
Copy link
Contributor Author

Guite commented Feb 9, 2020

what's the state of this PR? anything to do in order to get it merged?

@odolbeau
Copy link
Member

odolbeau commented Feb 9, 2020

Could you remove the non-trigerred PHPStan error & replace the service declaration with the code suggested by @rvanlaak please?

_instanceof:
    PhpParser\NodeVisitor:
        tags:
            - { name: 'php_translation.visitor', type: 'php' }

It will avoid the current Travis error when the bundle is used with a php-translation/extractor version which doesn't contain the new visitor.

It's done in #396 but this PR does several things. :/

@Guite
Copy link
Contributor Author

Guite commented Feb 9, 2020

@odolbeau done both but eventually this lead to including almost everything from #396 - except the autowire enablement.

@odolbeau
Copy link
Member

@Guite could you squash your commits please?
@rvanlaak WDYT?
It it OK for you if we merge this PR which is now ready to be merged or do you prefer to correct #396?

@Guite
Copy link
Contributor Author

Guite commented Feb 10, 2020

could you squash your commits please?

Please do so when clicking on the merge button.

@rvanlaak rvanlaak merged commit 50e54c5 into php-translation:master Feb 17, 2020
@welcoMattic welcoMattic mentioned this pull request Feb 21, 2020
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.

4 participants