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

[WIP] Clean up #4720

Merged
merged 5 commits into from Jan 7, 2018
Merged

[WIP] Clean up #4720

merged 5 commits into from Jan 7, 2018

Conversation

carusogabriel
Copy link
Contributor

@carusogabriel carusogabriel commented Dec 31, 2017

I've removed all white spaces and extra lines from our documentation 😄

@Naktibalda
Copy link
Member

Thanks, but all the files in docs/modules and docs/reference are auto-generated from .php files,
so your changes will be lost.
It would be better to fix the generator - please take a look at buildDocs method in Robofile.php

@carusogabriel
Copy link
Contributor Author

carusogabriel commented Dec 31, 2017

@Naktibalda Oh, okay. I’m gonna search how we can fix it, and amend this PR 👨‍💻

Thanks for guide me!

@carusogabriel
Copy link
Contributor Author

@Naktibalda Can you please tell me how do you run the RoboFile class to generate the docs? I guess I fixed the issues, but I couldn't find how to run it 😇

@Naktibalda
Copy link
Member

Download http://robo.li/robo.phar and run it in Codeception directory.

@carusogabriel carusogabriel changed the title Clean up [WIP] Clean up Jan 2, 2018
@DavertMik
Copy link
Member

Thanks! That looked like a minor issue the days when I created that build:docs script. But I didn't know how to handle it. After few years you finally improved that part. Thank you! If you have other ideas how to make docs better - I'm listening.

P.S. I hope that won't break anything on website :)

@DavertMik DavertMik merged commit 0f5c511 into Codeception:2.3 Jan 7, 2018
@carusogabriel carusogabriel deleted the clean-up branch January 7, 2018 23:48
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.

None yet

3 participants