Skip to content

Docs 📖: Add link to gitlab docs on obtaining a token#278

Merged
gpocentek merged 1 commit intopython-gitlab:masterfrom
asfaltboy:link-docs-gitlab-token
Aug 4, 2017
Merged

Docs 📖: Add link to gitlab docs on obtaining a token#278
gpocentek merged 1 commit intopython-gitlab:masterfrom
asfaltboy:link-docs-gitlab-token

Conversation

@asfaltboy
Copy link
Copy Markdown
Contributor

I find these sort of links very user friendly 😅

@asfaltboy asfaltboy changed the title Docs: Add link to gitlab docs on obtaining a token Docs 📖: Add link to gitlab docs on obtaining a token Jun 23, 2017
Copy link
Copy Markdown
Contributor

@gpocentek gpocentek left a comment

Choose a reason for hiding this comment

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

Very good idea, thank you. Can you update the sentence with my suggestions?

Thanks!

docs/cli.rst Outdated
- URL for the GitLab server
* - ``private_token``
- Your user token. Login/password is not supported.
Please refer `the official documentation`__ to learn how to obtain a token.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You can remove Please here as you're not asking for permission to the reader ;)

I'm not a native english speaker but I believe "Refer to the official doc..." is better.

I find these sort of links very user friendly 😅
@asfaltboy asfaltboy force-pushed the link-docs-gitlab-token branch from 5763741 to 9b8b806 Compare August 4, 2017 14:47
@asfaltboy
Copy link
Copy Markdown
Contributor Author

Thanks @gpocentek , removed "Please" and amended the commit.

@gpocentek
Copy link
Copy Markdown
Contributor

Thanks! The test failures are not related to this patch so let's merge!

@gpocentek gpocentek merged commit 657f011 into python-gitlab:master Aug 4, 2017
@asfaltboy asfaltboy deleted the link-docs-gitlab-token branch August 4, 2017 18:25
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.

2 participants