-
-
Notifications
You must be signed in to change notification settings - Fork 32
Commit
- Loading branch information
There are no files selected for viewing
8 comments
on commit 84d2678
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome job 👍 . I was looking for normal Symfony\Console integration twice this week.
People are still using the Router one. I'll recommend this everywhere.
I see you have separated Nette\DI compiler extension. Do you plan to add Symfony DI adapter as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you plan to add Symfony DI adapter as well?
What do you mean by this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@enumag @TomasVotruba Thank you, I appreciate it.
Do you plan to add Symfony DI adapter as well?
I don't known neither.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Symfony Bundle to allow loading Command the same way they are in Nette:
services:
- MyCommand
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@TomasVotruba I get your point now, but this repo is mainly for Nette. We'll see in the future.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@f3l1x Ok, thanks for into.
I might create a package to support Nette, Symfony and Laravel for now. I'm often switching framework and need 1 tool to use.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@TomasVotruba You know that should NOT be one package, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@enumag I know that's common idea. We can talk about it in person if you want.
What was the bug in 3.2.0?