Skip to content

Refactoring#3

Open
luspena wants to merge 11 commits into
kranonit:masterfrom
luspena:master
Open

Refactoring#3
luspena wants to merge 11 commits into
kranonit:masterfrom
luspena:master

Conversation

@luspena

@luspena luspena commented Feb 26, 2013

Copy link
Copy Markdown

WARNING!
First commit (fe3c90b) revert commit from merge with Grandis/master (15acf01).
Reason: need to work with not refactored sources.

@stokito

stokito commented Feb 26, 2013

Copy link
Copy Markdown
Member

Аня, правильно ли я понял, что твой пул реквест не отличается от того что сделал Кирилл? Ну кроме комментариев на английском :)

@luspena

luspena commented Feb 26, 2013

Copy link
Copy Markdown
Author

??????. ???? ?? ????? ?? ??????, ??? ??? ?????? ? ?, ?? ????? ??, ??
?????????? :)

26 ??????? 2013 ?., 20:25 ???????????? Sergey Ponomarev <
notifications@github.com> ???????:

???, ????????? ?? ? ?????, ??? ???? ??? ??????? ?? ?????????? ?? ???? ???
?????? ??????? ?? ????? ???????????? ?? ?????????? :)

Reply to this email directly or view it on GitHubhttps://github.com//pull/3#issuecomment-14130851
.

@stokito

stokito commented Feb 26, 2013

Copy link
Copy Markdown
Member

глянь кодировку у комментария

@luspena

luspena commented Feb 26, 2013

Copy link
Copy Markdown
Author

Опс, прошу прощения. Ответ непосредственно с gmail не подхватился.

Привет. Если он пилил по книжке, как это делала и я, то думаю да, не отличается :)

@stokito

stokito commented Feb 26, 2013

Copy link
Copy Markdown
Member

ок, просто я не смотрел историю комитов.
второй вопрос: делала ли ты юнит тесты?

@luspena

luspena commented Feb 26, 2013

Copy link
Copy Markdown
Author

Да, к существующему добавила тест на версию с html

@stokito

stokito commented Feb 26, 2013

Copy link
Copy Markdown
Member

ну... кагбе хотелось бы побольше, но всё ранво умничка :) 👍

@luspena

luspena commented Feb 26, 2013

Copy link
Copy Markdown
Author

Спасибо. Побольше - эт хорошая идея :)

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