Skip to content

Conversation

@vanyasem
Copy link
Contributor

@vanyasem vanyasem commented Nov 24, 2017

Я немного поздно, но вот тут код с нормальными отступами. Не требует изменений в уже существующих программах. А, и никаких Леонидов в переменных ;)

@@ -1,5 +1,4 @@
# -*- coding: utf-8 -*-
import collections
Copy link
Contributor Author

Choose a reason for hiding this comment

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

ненужный импорт висел с прошлого ПРа, забыл убрать

)

return tstr
return tstr[:-151]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

:151 убирает "мусор" на 151 символ, который остаётся после ссылки

Copy link
Owner

Choose a reason for hiding this comment

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

@vanyasem а оно всегда 151? там у вк тоже 151 символ обрезается?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

а хер знает, если честно ) на нескольких альбомах прогнал - работает. могу убрать, это просто query params после ссылки, ни на что не влияют в любом случае

Copy link
Contributor

Choose a reason for hiding this comment

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

А она настолько большая что ее стоит обрезать? Просто в любом случаи лучше ввести себя как клиент вк

Copy link
Contributor Author

@vanyasem vanyasem Nov 24, 2017

Choose a reason for hiding this comment

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

ну в ней 151 символ, неприятно. если претензии - верну обратно. чисто эстетика

Copy link
Owner

Choose a reason for hiding this comment

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

Давай лучше уберем тогда

@vanyasem
Copy link
Contributor Author

Всё, мерджим )

@python273 python273 merged commit 13149b9 into python273:master Nov 24, 2017
@python273
Copy link
Owner

👍

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.

3 participants