Skip to content

Conversation

@cacapon
Copy link
Owner

@cacapon cacapon commented May 6, 2025

fixed #215

対応内容

今までの対応で'$SHELL'のようなシングルクオートで囲まれていた文字が展開されるようになってしまったので、
展開されないように修正をしています。

現時点の対応だとhe's cool!のような閉じていない文字でsyntax errorになってしまうので、対応中です。
対応しました。レビュー可能です。

確認方法

./minishで確認できます。

@cacapon cacapon self-assigned this May 6, 2025
@cacapon cacapon added the WIP まだ作業が完了していないプルリクなどに label May 6, 2025
@cacapon cacapon marked this pull request as draft May 6, 2025 06:40
Copy link
Collaborator

@dayano74 dayano74 left a comment

Choose a reason for hiding this comment

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

展開されないことを確認しました。大丈夫だと思います。

dayano74
dayano74 previously approved these changes May 6, 2025
@dayano74
Copy link
Collaborator

dayano74 commented May 6, 2025

approveしてもマージされないのはwipされているからですか?

@cacapon
Copy link
Owner Author

cacapon commented May 6, 2025

はい、今ドラフトで作ってるからですね。
Discordにも書いていますが対応がまだ途中なんです。(閉じてないクオートでエラーになるバグが出た)

これは今回の対応で新しく出たバグなので、その対応が終わったら改めてレビューさせてください。

@cacapon cacapon removed the WIP まだ作業が完了していないプルリクなどに label May 6, 2025
@cacapon cacapon requested a review from dayano74 May 6, 2025 11:51
@cacapon cacapon marked this pull request as ready for review May 6, 2025 11:51
Copy link
Collaborator

@dayano74 dayano74 left a comment

Choose a reason for hiding this comment

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

3パターンの形式を複数試しました。大丈夫だと思います!修正ありがとうございます。

echo $SHELL
echo "$SHELL"
echo '$SHELL'

@dayano74 dayano74 merged commit c911276 into main May 9, 2025
2 checks passed
@dayano74 dayano74 deleted the feature/215-fix-expand-env branch May 9, 2025 04:49
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.

シングルクオートでも環境変数が展開されてしまう。

3 participants