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

Add X-Ray integration #32

Open
marcioAlmada opened this issue Sep 23, 2016 · 1 comment
Open

Add X-Ray integration #32

marcioAlmada opened this issue Sep 23, 2016 · 1 comment

Comments

@marcioAlmada
Copy link
Owner

marcioAlmada commented Sep 23, 2016

Engine integration should come through a small php extension bridge instead:

https://github.com/marcioAlmada/xray

Reasons:

  • hijacking the composer auto loader and passing things through a stream wrapper is not sustainable enough.
  • using __halt_compiler as suggested in Real Time Mode #11 was a brilliant hack but it's a PITA for the end users.
  • an engine extension can provide syntax check at preprocessor time effortlessly, even for global macros - aka not localized in the file being linted.
@chris-kruining
Copy link

Is an extension the only viable solution at this point? I ask because unlike packages there is no proper package manager for extensions. If I'm wrong, please tell me, such a manager would solve a lot of issues for me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants