Skip to content
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

Reorganize the folder structure and apply new CS standard #405

Conversation

ste93cry
Copy link
Collaborator

@ste93cry ste93cry commented Jan 4, 2021

As per title, I've aligned both the CS standard and the folder structure with the one used in the main SDK. The only notable change is the introduction of declare(strict_types=1) in all files. I also deleted the whole examples folder as I judged it useless, unmaintained, and because I think it's better to improve the documentation rather than providing yet another copy of a basic Symfony app that I don't understand how it can help new users

@ste93cry ste93cry added this to the 4.0 milestone Jan 4, 2021
@ste93cry ste93cry requested a review from Jean85 January 4, 2021 13:00
@ste93cry ste93cry force-pushed the reorganize-folder-structure-and-apply-new-cs-standard branch 2 times, most recently from 54eb3d6 to 07beb86 Compare January 4, 2021 13:22
@Jean85
Copy link
Collaborator

Jean85 commented Jan 4, 2021

I also deleted the whole examples folder as I judged it useless, unmaintained, and because I think it's better to improve the documentation rather than providing yet another copy of a basic Symfony app that I don't understand how it can help new users

I agree: the End2End tests serve the same purposes and are automatically aligned with the code.

@ste93cry ste93cry force-pushed the reorganize-folder-structure-and-apply-new-cs-standard branch from 07beb86 to 9c76713 Compare January 4, 2021 18:08
phpunit.xml Outdated Show resolved Hide resolved
@ste93cry ste93cry force-pushed the reorganize-folder-structure-and-apply-new-cs-standard branch from 4714aee to 8625fbf Compare January 5, 2021 11:40
@ste93cry ste93cry requested a review from Jean85 January 5, 2021 11:41
@Jean85 Jean85 merged commit 7f82f1c into getsentry:master Jan 5, 2021
@ste93cry ste93cry deleted the reorganize-folder-structure-and-apply-new-cs-standard branch January 5, 2021 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants