Skip to content

Conversation

@y8
Copy link
Contributor

@y8 y8 commented Feb 23, 2014

Мержить пока не надо, просто чтоб ты видел то, что мы обсуждаем :)

y8 added 4 commits February 23, 2014 20:00
Выпиливаем $KCODE и разбиваем документ на текстовые ноды, над которыми
уже начинаем издеваться.
@y8
Copy link
Contributor Author

y8 commented Feb 23, 2014

По поводу перехода на юникод, да, почему бы и нет.

@julik
Copy link
Owner

julik commented Feb 24, 2014

Оч круто. С ассертами - у меня там есть assert_equal_cp, он удобный. Дальше: мне кажется целесообразным в коде самого гиленсона - не в тестах - ставить хитрые unicode-символы через массив. Тогда можно набрать их кодпойнты через 0x123, что соответствует их номенклатуре во всех таблицах и легко гуглится например. Плюс inline после такого глифа можно поставить комментарий с его официальным названием. Это реально неслабо помогает понять что же там имеется в виду. Как-то так: https://github.com/julik/gilenson/blob/master/lib/gilenson.rb#L119
мне это кажется актуальным особенно для whitespace (которое white и его не видно). В тестах можно я думаю ставить эти штуки inline но помечать рядом с ассерцией что "здесь зашит ТАКОЙ ПРОБЕЛ о котором вы даже НЕ ПОДОЗРЕВАЛИ".

Про jeweler - я хочу чтобы было две вещи:

  • bundle exec rake (без аргументов и шоп работало)
  • bundle exec rake release (без аргументов и шоп работало)

и чтобы release делал тег и пушил его, сам собирал гем и сам его публиковал. Я не в курсе умеют ли последние rubygems/bundler все это сами делать. И делать все это руками мне неудобно (у меня гемов один маленький фургон, и все они релизятся и тестятся так, и помнить что где-то кто-то из нелюбви к зависимостям это выпилил меня не радует). Это обеспечивается?

@julik
Copy link
Owner

julik commented Oct 5, 2014

Ну как там движуха, заглохла совсем?

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.

2 participants