Skip to content

Introduce doc generation tools #18

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.

Sign up for GitHub

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

Jump to bottom
Merged
dwsupplee merged 4 commits into from
Mar 1, 2016
Merged

Introduce doc generation tools #18

dwsupplee merged 4 commits into from
Mar 1, 2016

Conversation

@dwsupplee
Copy link
Contributor

@dwsupplee dwsupplee commented Feb 19, 2016

@dwsupplee dwsupplee added documentation do not merge Indicates a pull request not ready for merge, due to either quality or timing. labels Feb 19, 2016
@callmehiphop
Copy link
Contributor

callmehiphop commented Feb 19, 2016

👍 is there a demo?

@dwsupplee
Copy link
Contributor Author

dwsupplee commented Feb 19, 2016

Working on getting a pretty rough demo of the gh-pages up right now

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 19, 2016
tmatsuo
tmatsuo reviewed Feb 19, 2016
View reviewed changes
scripts/DocGenerator.php
@@ -0,0 +1,233 @@
<?php

This comment was marked as spam.

Sign in to view

@dwsupplee
Copy link
Contributor Author

dwsupplee commented Feb 22, 2016

WIP version of the docs site for anyone interested :)

http://dwsupplee.github.io/gcloud-php/#/

@JustinBeckwith
Copy link
Contributor

JustinBeckwith commented Feb 22, 2016

looking good!

@dwsupplee dwsupplee removed do not merge Indicates a pull request not ready for merge, due to either quality or timing. in progress labels Feb 26, 2016
bshaffer
bshaffer reviewed Feb 26, 2016
View reviewed changes
composer.json Outdated
"autoload": {
"psr-4": {
"Google\\Cloud\\": "src"
"Gcloud\\": "src"

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

dwsupplee added a commit that referenced this pull request Mar 1, 2016
@dwsupplee
Merge pull request #18 from dwsupplee/docs
7d2fd94 
Introduce doc generation tools
@dwsupplee dwsupplee merged commit 7d2fd94 into googleapis:master Mar 1, 2016
@tmatsuo
Copy link
Contributor

tmatsuo commented Mar 1, 2016

Nice work! Just a small nit, the demo still says composer install google/cloud, but now it should be google/gcloud? I understand that it is just a demo, but I'm wondering where it comes from.

@dwsupplee
Copy link
Contributor Author

dwsupplee commented Mar 1, 2016

Thanks @tmatsuo!

Good eye. You're correct it should be google/gcloud, I'll make sure it gets corrected.

@cedricziel
Copy link
Contributor

cedricziel commented Mar 1, 2016

The example on the frontpage has an issue with escaping

@dwsupplee
Copy link
Contributor Author

dwsupplee commented Mar 1, 2016

Thanks @cedricziel. It appears there may be a configuration issue with hljs (the syntax highlighting library our docs site uses). Working on it now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

cla: yes This human has signed the Contributor License Agreement. documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@dwsupplee @callmehiphop @JustinBeckwith @tmatsuo @cedricziel @bshaffer @googlebot