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: small documentation review #1516

Merged
merged 1 commit into from
May 1, 2017

Conversation

ldez
Copy link
Contributor

@ldez ldez commented Apr 30, 2017

Description

  • fix syntax coloration.
  • same language, same code fence.
  • fix typo.
  • add some magic.

@ldez ldez added this to the 1.3 milestone Apr 30, 2017
@ldez ldez force-pushed the refactor/documentation-review branch from 1cb4d82 to 5c1cdd3 Compare April 30, 2017 18:24
@ldez ldez changed the title refactor: small documentation review doc: small documentation review Apr 30, 2017
docs/basics.md Outdated
[frontends.frontend2]
[frontends.frontend2.routes.test_1]
rule = "Host:test1.localhost,test2.localhost"
[frontends.frontend3]
Copy link
Contributor

Choose a reason for hiding this comment

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

Extra leading space?

annotation. By default if the annotation is not set at all Træfik will include the
ingress. If the annotation is set to anything other than traefik or a blank string
Træfik will ignore it.


![](http://i.giphy.com/ujUdrdpX7Ok5W.gif)
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure why, but still ☺️

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

🤣

@ldez ldez force-pushed the refactor/documentation-review branch from 5c1cdd3 to 5526028 Compare April 30, 2017 18:56
Copy link
Contributor

@timoreimann timoreimann left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Member

@emilevauge emilevauge left a comment

Choose a reason for hiding this comment

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

Thanks @ldez
LGTM

@ldez ldez force-pushed the refactor/documentation-review branch from 5526028 to 4c4b05d Compare May 1, 2017 16:09
@emilevauge emilevauge merged commit bec45bc into traefik:master May 1, 2017
@ldez ldez deleted the refactor/documentation-review branch May 1, 2017 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants