-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Make Twig services and event listeners private #108
Conversation
Listeners cannot be private. they are access dynamically to lazy-load them (you should be getting an exception during the container building btw). @javiereguiluz what about enabling Travis on the repo to run the tests ? |
Thanks @bocharsky-bw for sending PRs, reviewing code and being so active on this repository! |
This PR was merged into the master branch. Discussion ---------- Make Twig services and event listeners private Thanks to stof in #104 (comment) Commits ------- 129960d Make Twig services and event listeners private
The markdown service should be private too btw, for the same reason than Twig extensions (it is only used as a dependency of the twig extension) |
@javiereguiluz you merged a broken PR |
You are right! It was fixed on 3608348 |
Travis is set up in the |
@fabpot can you enable Travis on this repo ? |
@stof and @javiereguiluz thank you! |
@stof done |
Thanks to stof in #104 (comment)