Skip to content

Conversation

@mikoto2000
Copy link

@mikoto2000 mikoto2000 commented Nov 25, 2025

#836 の実装を行っています。
現時点でツッコミどころなど、指摘いただけると幸いです。

TODO:

  • : Neovim 対応
  • : OS でなく、 shell の種類でオプションを分ける(linux shell : -c or pwsh or powershell : -C or cmd.exe : /c)
  • : Windows での文字化け対応
  • : doc

@mikoto2000 mikoto2000 marked this pull request as ready for review November 30, 2025 02:49
@mikoto2000 mikoto2000 changed the title WIP: Added System.AsyncProcess. Added System.AsyncProcess. Dec 1, 2025
@tsuyoshicho
Copy link
Contributor

tsuyoshicho commented Dec 1, 2025

draft 時は1コミットずつテストOKしないとだめだった、たぶん今は平気

だめだった……再度実行はしました

Comment on lines 101 to 107
" v8.2.238 or earlier, job_stop() exit status is 0.
" See: https://github.com/vim/vim/commit/b3e195cca7b3201b188c1713b64012b1bef4f61f
if v:version == 802 && !has('patch238') || v:version < 802
Assert True(g:exit_code == 0)
else
Assert True(g:exit_code != 0)
endif
Copy link
Contributor

Choose a reason for hiding this comment

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

えっと、パッチされたのは Windows だけな気がするので、Windowsとそれ以外では違う処理にしないとではないかと

Copy link
Author

Choose a reason for hiding this comment

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

ご指摘の通り...
コミット 1e9054b で修正しました。

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.

2 participants