Skip to content

Conversation

@dayano74
Copy link
Collaborator

#257 のテストケースは通るようになったので、検証してほしいです。

ちなみに、 echo hello > out > >のようなコマンドが入力された時は、parserで弾く認識で合っていますか?

@dayano74 dayano74 requested a review from cacapon May 15, 2025 12:29
@dayano74 dayano74 linked an issue May 15, 2025 that may be closed by this pull request
@cacapon
Copy link
Owner

cacapon commented May 16, 2025

ちなみに、 echo hello > out > >のようなコマンドが入力された時は、parserで弾く認識で合っていますか?

はい、そこはパーサーで弾きます

Copy link
Owner

@cacapon cacapon left a comment

Choose a reason for hiding this comment

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

OKです!
コードも簡潔に解決できていて素晴らしいです。ありがとうございました!

@cacapon cacapon merged commit 747aa48 into main May 16, 2025
2 checks passed
@cacapon cacapon deleted the 257-fix-redirect-process branch May 16, 2025 02:03
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.

fix redirect process

3 participants