Skip to content

Fix style check#29212

Merged
tavplubix merged 1 commit intomasterfrom
fix_style_check
Sep 21, 2021
Merged

Fix style check#29212
tavplubix merged 1 commit intomasterfrom
fix_style_check

Conversation

@tavplubix
Copy link
Copy Markdown
Member

@tavplubix tavplubix commented Sep 21, 2021

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label Sep 21, 2021
@alesapin
Copy link
Copy Markdown
Member

alesapin commented Sep 21, 2021

We shouldn't commit autogenerated files.

@alesapin alesapin self-assigned this Sep 21, 2021
@azat azat mentioned this pull request Sep 21, 2021
Merged
@tavplubix
Copy link
Copy Markdown
Member Author

tavplubix commented Sep 21, 2021

Docs check — Found 2 errors - looks irrelevant:

INFO:root:Running tests for ru
INFO:root:Tag <a class="alert-link" href="#-company-md">contact us</a>
INFO:root:Link to nowhere: #-company-md
INFO:root:Tag <a class="alert-link" href="#-company-md">contact us</a>
INFO:root:Link to nowhere: #-company-md
ERROR:root:Found 2 links to nowhere in ru
INFO:root:Running tests for en
INFO:root:wkhtmltopdf --print-media-type --log-level warn /tmp/1079819988/tmpn9a_xl1h/single/index.html /place/sandbox-data/tasks/8/8/1079819988/ClickHouse/docs/build/docs/ru/single/clickhouse_ru.pdf
INFO:root:Tag <a class="alert-link" href="#-company-md">contact us</a>
INFO:root:Link to nowhere: #-company-md
INFO:root:Tag <a class="alert-link" href="#-company-md">contact us</a>
INFO:root:Link to nowhere: #-company-md
ERROR:root:Found 2 links to nowhere in en

@tavplubix tavplubix merged commit b872462 into master Sep 21, 2021
@tavplubix tavplubix deleted the fix_style_check branch September 21, 2021 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alesapin alesapin alesapin approved these changes

Assignees

@alesapin alesapin

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tavplubix @alesapin @robot-clickhouse