Skip to content

Conversation

@Fatalll
Copy link
Owner

@Fatalll Fatalll commented Feb 24, 2019

No description provided.

* в execute теперь можно передать IStorage, рекомендуется через него хранить окружение
@Fatalll Fatalll changed the title Первая домашняя работа Домашняя работа 1 Mar 1, 2019
Copy link

@ottergottaott ottergottaott left a comment

Choose a reason for hiding this comment

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

Работа очень хорошая, но есть некоторые недоделки.

  • echo asd | cat выводит cat: must specify file names!, ожидается, что выведет asd
  • pwd не работает, т.к. пакет os не импортирован:
  • x = 1 \ echo " '$x' " выводит $x, ожидается '1'
  • вызов внешних программ работает, но не передает аргументы во внешнюю команду. например git status выведет help, как если бы позвали просто git

Отдельный большой плюсик за использование type hints.

@ottergottaott
Copy link

А, да. Было бы еще хорошо сделать нормальные packages

@Fatalll
Copy link
Owner Author

Fatalll commented Jun 8, 2019

Работа очень хорошая, но есть некоторые недоделки.

  • echo asd | cat выводит cat: must specify file names!, ожидается, что выведет asd

Исправил

  • pwd не работает, т.к. пакет os не импортирован:

Исправил

  • x = 1 \ echo " '$x' " выводит $x, ожидается '1'

Поменял приоритет и регулярку, вроде теперь хорошо:
image

  • вызов внешних программ работает, но не передает аргументы во внешнюю команду. например git status выведет help, как если бы позвали просто git

У меня вроде все работает,
image

@Fatalll
Copy link
Owner Author

Fatalll commented Jun 8, 2019

А, да. Было бы еще хорошо сделать нормальные packages

в task-2 сделал и заодно добавил CI :) Сюда уж не стал мерджить, т.к. поедет все.

@ottergottaott
Copy link

Ок, @yurii-litvinov думаю, что можно засчитывать

@yurii-litvinov
Copy link

Окей, зачтена

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.

4 participants