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

Are necessary any implicit changes made by GitLocalize? #204

Open
buslov opened this issue Jan 10, 2018 · 4 comments
Open

Are necessary any implicit changes made by GitLocalize? #204

buslov opened this issue Jan 10, 2018 · 4 comments

Comments

@buslov
Copy link
Contributor

buslov commented Jan 10, 2018

GitLocalize adds some changes when making a pull-request:

  • Replaces * by - in lists
    image
  • Extra empty lines
    image
  • Completely breakes TOC's links
    image
  • Deletes redundant backquotes
    image
  • Changes formatting characters:
    image
  • Decodes (some?) HTML entities:
    image
@RichardLitt
Copy link
Collaborator

@sotayamashita these changes seem out of scope for a translation engine. Are they necessary?

@sotayamashita
Copy link
Contributor

sotayamashita commented Jan 11, 2018

@RichardLitt GitLocalize changes some markdown following flow. In GitLocalize, we parse markdown and convert it into HTML to translate. After translating, we convert HTML to markdown based on commonmark spec. A series of the flow, we change some markdown.

We understand that issue but it will rendered same before. Is there a problem?

Thanks in advance. 🙇

@RichardLitt
Copy link
Collaborator

Good to know. You're right, most of these are superficial. However, the TOC changes aren't - as far as I know, they will break the links. @buslov can you confirm this?

@buslov
Copy link
Contributor Author

buslov commented Jan 11, 2018

@RichardLitt, you are right. I suppose that GitLocalize just copies links from English source. We could add some tests (#205) to Travis CI to prevent any merges with broken links.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants