-
-
Notifications
You must be signed in to change notification settings - Fork 414
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 installer for TastyIgniter extensions and themes #478
Conversation
@niksamokhvalov here's the new PR :) |
README.md
Outdated
@@ -115,6 +115,7 @@ is not needed to install packages with these frameworks: | |||
| Sylius | `sylius-theme` | |||
| symfony1 | **`symfony1-plugin`** | |||
| TAO | `tao-extension` | |||
| TastyIgniter | **`igniter-extension`<br>`igniter-theme`** |
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.
I think, that we should use the full name for types to avoid confusion: tastyigniter-extension
and tastyigniter-theme
. Just as it is in CodeIgniter. Can you fix it? Also need to rename IgniterInstaller
and IgniterInstallerTest
classes.
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.
Done
No description provided.