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

Uses ASCII characters to spell Traefik #4063

Merged
merged 5 commits into from Oct 17, 2018

Conversation

geraldcroes
Copy link
Contributor

@geraldcroes geraldcroes commented Oct 17, 2018

What does this PR do?

This PR does something terrible ... but ... it's probably better that way (at least let us believe that, it'll be easier to accept).

This PR replaces the spelling of our beloved Træfik with Traefik (yes, only plain old boring ascii characters).

Motivation

Oh believe me that there is no desire nor willingness to do so, I'm thinking "Good Bye Old Friend" right now, but it'll be easier to stay consistent in the documentation, comments, toml, code, articles, and so on.

More

  • Added/updated tests
  • Added/updated documentation

Additional Notes

The former æ will stay in the logo (and in our ❤️ )!

Copy link
Member

@mmatur mmatur left a comment

Choose a reason for hiding this comment

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

Since you have replace æ by ae you have broke some links

docs/user-guide/kv-config.md Outdated Show resolved Hide resolved
docs/user-guide/kv-config.md Outdated Show resolved Hide resolved
Copy link
Member

@mmatur mmatur 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

@jbdoumenjou jbdoumenjou left a comment

Choose a reason for hiding this comment

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

LGTM We will not forget you æ ❤️

Copy link
Member

@mmatur mmatur left a comment

Choose a reason for hiding this comment

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

Do we have to update CHANGELOG.md?

@emilevauge
Copy link
Member

emilevauge commented Oct 17, 2018

@mmatur I don't think this is necessary to update the changelog, but as it has already been modified, I'm OK to keep it.

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.

👋 æ
LGTM

@mmatur
Copy link
Member

mmatur commented Oct 17, 2018

@geraldcroes Could you please revert the change in the CHANGELOG.md file

Copy link
Member

@mmatur mmatur left a comment

Choose a reason for hiding this comment

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

LGTM

@errm
Copy link
Contributor

errm commented Oct 18, 2018

💔 👋 æ

@ldez ldez deleted the feature/traefik-spelling branch October 18, 2018 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants