-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Allow license.php for app repos aswell #22340
Conversation
By analyzing the blame information on this pull request, we identified @DeepDiver1975, @LukasReschke and @MorrisJobke to be potential reviewers |
I see this which is caused by the git blame
|
$buildDir = getcwd(); | ||
if ($gitRoot) { | ||
chdir($gitRoot); | ||
$file = substr($file, strlen($gitRoot) + 1); |
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.
removing the +1 fixes my issue
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.
ah, I ran apps/appname
, you ran apps/appname/
will make it work for both.
Handling vendor code within the app also seems to have issues
|
f2ddbc1
to
100b357
Compare
tested with the mail app and works here now locally |
👍 |
1 similar comment
👍 |
…s-aswell Allow license.php for app repos aswell
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Before
cc @DeepDiver1975 @MorrisJobke @LukasReschke