Skip to content

Conversation

@Fatalll
Copy link
Owner

@Fatalll Fatalll commented Mar 1, 2019

No description provided.

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.

В целом очень хорошо, но

  • Не распозанет ошибку в такой команде grep -A -1 text test.txt и работает, как обычно, хотя -1 не является валидным аргументом.

Еще на будущее: старайся делать pr, в котором есть только актуальные для него коммиты. В данном случае это можно было бы сделать создав pr не в ветку master, а в task-1

@Fatalll
Copy link
Owner Author

Fatalll commented Jun 8, 2019

  • Не распозанет ошибку в такой команде grep -A -1 text test.txt и работает, как обычно, хотя -1 не является валидным аргументом

Действительно, добавил проверку на значения меньше или равные 0 для A, остальные ошибки должен выдавать корректно.

@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