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

php parser dependency #244

Closed
flip111 opened this issue Jun 16, 2015 · 9 comments
Closed

php parser dependency #244

flip111 opened this issue Jun 16, 2015 · 9 comments
Labels

Comments

@flip111
Copy link
Contributor

flip111 commented Jun 16, 2015

Is it possible to update the dependency on php parser? 0.9.4 is required while 1.3.0 is already out. In particular i'm getting composer conflicts when also trying to install this package https://github.com/Ocramius/GeneratedHydrator/blob/master/composer.json#L25

@skotaa
Copy link

skotaa commented Jun 24, 2015

We created pull request to resolve this issue, if you want you can use our changes from our repository: https://github.com/domenypl/JMSTranslationBundle, tag 1.1.1

@flip111
Copy link
Contributor Author

flip111 commented Jun 24, 2015

Oh nice, though i could only find tag 1.1 and not 1.1.1

@skotaa
Copy link

skotaa commented Jun 26, 2015

Probably you do not have correctly configured Composer (composer.json). Did you add the repository: "https://github.com/domenypl/JMSTranslationBundle.git" ?

@flip111
Copy link
Contributor Author

flip111 commented Jun 26, 2015

I just meant when looking at github

image

@Astinus-Eberhard
Copy link

C'mon, read carefully: tag, not branch...

bez nazwy
bez nazwy

@flip111
Copy link
Contributor Author

flip111 commented Jun 30, 2015

I never noticed this extra tab on github. Thank you

@moroine
Copy link

moroine commented Aug 11, 2015

I really need this feature as I use php 5.5 with keywords self, static ...

So the PhpParser v0.9.0 it's now outdated as it can't parse my variables static::class firing unexpected T_CLASS ...

So I used the fork: https://github.com/domenypl/JMSTranslationBundle which work nicely
Thank you

@sagikazarmark
Copy link

Duplicate of #155

@Nyholm
Copy link
Collaborator

Nyholm commented Aug 4, 2016

The parser is updated.

@Nyholm Nyholm closed this as completed Aug 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants