New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change regular expression for adding sudo with exclamation. #220

Merged
merged 1 commit into from Jul 22, 2013

Conversation

Projects
None yet
2 participants
@a3no
Contributor

a3no commented Jul 22, 2013

sudoを追加する際に否定演算子を考慮するように正規表現を変更しました。
否定演算子はDebian系のbe_installedで使用されているのを確認しています。

@mizzy mizzy merged commit d27be6f into mizzy:master Jul 22, 2013

1 check passed

default The Travis CI build passed
Details
@mizzy

This comment has been minimized.

Show comment
Hide comment
@mizzy

mizzy Jul 22, 2013

Owner

ありがとうございます!

頂いた修正で問題なかったのですが、!は存在しないか1つだけあるか、のどちらかの状態で、2つ以上続くケースはなさそうなので、!*!?に修正したのと、!の場合だけではなく(の場合も、後ろの空白文字を同様に扱った方が良さそうだと思いましたので、¥s*の位置を後ろにずらしました。

それから、今後このあたりのテストがしやすいように、別途テストを追加しました。

v0.7.1としてリリース済みです。

プルリクエストありがとうございました。今後ともよろしくお願いします。

Owner

mizzy commented Jul 22, 2013

ありがとうございます!

頂いた修正で問題なかったのですが、!は存在しないか1つだけあるか、のどちらかの状態で、2つ以上続くケースはなさそうなので、!*!?に修正したのと、!の場合だけではなく(の場合も、後ろの空白文字を同様に扱った方が良さそうだと思いましたので、¥s*の位置を後ろにずらしました。

それから、今後このあたりのテストがしやすいように、別途テストを追加しました。

v0.7.1としてリリース済みです。

プルリクエストありがとうございました。今後ともよろしくお願いします。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment