-
Notifications
You must be signed in to change notification settings - Fork 483
fix(PSR-12): closing ?> tag MUST be omitted #2594
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
Conversation
Signed-off-by: Marc-Etienne Vargenau <[email protected]>
Signed-off-by: Marc-Etienne Vargenau <[email protected]>
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.
LGTM
@vargenau Thank you for the contribution. |
Create endpoints for CX operations (email, ecc, keywords, etc.) and use common functions in CopyrightController. Signed-off-by: Gaurav Mishra <[email protected]>
Info on 88d04ec BREAKING CHANGE: Response for `/uploads/{id}/item/{itemId}/copyrights` endpoint `#/components/schemas/GetFileCopyrights` parameter `copyright_count` is renamed to `count` to enable reusability by other CX endpoints. Signed-off-by: Gaurav Mishra <[email protected]>
Signed-off-by: Shaheem Azmal M MD <[email protected]>
Hey @vargenau , looks like you've merged with master branch instead. Please use |
This pull request has conflicts, please rebase with master to resolve those before we can evaluate the pull request. |
Sorry, I created a mess when tying to squash. I have created a new pull request #2599 Let us close this one. |
Merged #2599, thus closing this PR. |
Description
PSR-12:
All PHP files MUST end with a non-blank line, terminated with a single LF.
The closing ?> tag MUST be omitted from files containing only PHP.
Changes
Remove closing ?> tag