-
Notifications
You must be signed in to change notification settings - Fork 16
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 support for mounting another castor app #367
Conversation
lyrixx
commented
Apr 2, 2024
•
edited
edited
I like the approach, and it seems to meet the need for an issue I opened some time ago! :) |
Love it, made a proposition, it's only a namespace isn't it ? |
I like the idea 👍 |
01719f9
to
b7faa34
Compare
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.
Okay, the PR is ready for review
import(__DIR__ . '/tools/php-cs-fixer/castor.php'); | ||
import(__DIR__ . '/tools/phpstan/castor.php'); | ||
import(__DIR__ . '/tools/static/castor.php'); |
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.
For the record, theses "app" cannot be mounted, since they refer to configuration in at the root (see the doc)
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.
Some nitpicks but it looks good 🙌