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

doc: Fixed the picture on page http://ceph.com/docs/master/cephfs/ #5012

Merged
1 commit merged into from Jun 19, 2015

Conversation

Arshanes
Copy link
Contributor

Fixes: #9174

Signed-off-by Valentin Thomas valentin.arshanes.thomas@gmail.com

@ghost ghost added the documentation label Jun 18, 2015
@ghost ghost self-assigned this Jun 18, 2015
@ghost
Copy link

ghost commented Jun 18, 2015

https://raw.githubusercontent.com/Arshanes/ceph/8385391beb4e53c3644d386b526aa3cff64e4dc9/doc/cephfs/index.rst shows space differences. Could you please make sure you're only using spaces and no tabs ? That's the easiest way to ensure it's properly formatted. Otherwise looks great :-)

@Arshanes
Copy link
Contributor Author

I removed the unexpected tabs. I didn't see those 3 remaining. It's ok now. :)

@ghost
Copy link

ghost commented Jun 18, 2015

looks great :-) Could you please squash the two commits into one ?

Fixes: ceph#9174

Signed-off-by Valentin Thomas <valentin.arshanes.thomas@gmail.com>
@Arshanes
Copy link
Contributor Author

Commits are squashed. :)

ghost pushed a commit that referenced this pull request Jun 19, 2015
doc: Fixed the picture on page http://ceph.com/docs/master/cephfs/

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost ghost merged commit 160201f into ceph:master Jun 19, 2015
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants