Skip to content

Conversation

@Jean85
Copy link
Contributor

@Jean85 Jean85 commented Jan 24, 2019

This is just the start of the work toward 3.0. A lot more options are to be mapped. I'll open other issues in the same milestone to track other missing steps.

@Jean85 Jean85 added this to the 3.0 (Sentry client 2.0) milestone Jan 24, 2019
@Jean85 Jean85 mentioned this pull request Jan 24, 2019
This command requires you to have Composer installed globally, as explained
in the [installation chapter](https://getcomposer.org/doc/00-intro.md)
of the Composer documentation.
> TODO: Flex recipe
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Delete or fill ^^

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

prefixes: ~
release: ~
```
TODO
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would really just write basic docs here and move the majority to
https://github.com/getsentry/sentry-docs/tree/master/src/collections/_documentation/platforms/php symfony.md

@Jean85
Copy link
Contributor Author

Jean85 commented Feb 22, 2019

@HazAT I've addressed your review; I would like to address all the other comments in separated PRs, since that would need the SDK package release.

@Jean85 Jean85 mentioned this pull request Feb 22, 2019
HazAT and others added 3 commits February 22, 2019 09:49
Co-Authored-By: Jean85 <alessandro.lai85@gmail.com>
@Jean85 Jean85 mentioned this pull request Feb 25, 2019
Copy link
Collaborator

@stayallive stayallive left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm going to assume @HazAT did a good job with this review 😄

@Jean85 Jean85 merged commit 6f7d6cd into master Feb 25, 2019
@Jean85 Jean85 deleted the 3.0 branch February 25, 2019 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants