-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
Prepare docs for Doctrine website #34
Conversation
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.
I'm going to approve this for now. But there are a number of grammatical errors throughout the docs which should be corrected. I'm willing to do a PR on the docs as a follow-up to this PR to address them. I'm more concerned about getting this in place on the website.
Please create an issue, assigned to me, to review the docs' English as a follow-up to approving this PR.
If you want me to push a PR to this PR before you merge, I'm OK with that. I'll leave the decision for when to merge up to @driehle |
@TomHAnderson I'll merge this for now as this will likely enable me to render |
Similar to doctrine/DoctrineModule#733, this PR prepares the docs for being rendered on the Doctrine website. The README has been split into multiple sections and converted from .md to .rst. A .doctrine-project.json is added.
Similar to and in line with: