:raising_hand: PR to add to the README - docker re...
# proj-agent-canvas
g
šŸ™‹ PR to add to the README • docker requirements • docker code block for powershell • warning for "from source" https://github.com/OpenHands/agent-canvas/pull/945
w
IMO we should keep powershell instructions out of the main README, and put them in a separate doc
otherwise LGTM!
g
My only concern if we put windows terminal instruction in another readme is will it get updated as the readme gets updated? I put PowerShell/windows terminal changes in the main README at the start of AC and it got removed I just worry about ignoring Windows users like we have in the past.
w
good point on updates. maybe we can add code review instructions to make sure openhands catches that? I don't typically see powershell instructions in other READMEs, and a verbose README with lots of code blocks is often a turnoff for me
nod 1
g
good point on updates. maybe we can add code review instructions to make sure openhands catches that?
nod
I don't typically see powershell instructions in other READMEs, and a verbose README with lots of code blocks is often a turnoff for me
sounds good sg I will make adjustments to the PR
c
Oops my agent made it worse on the Windows issue, sorry! šŸ™
šŸ‘€ 1
g
I just pushed the changes request by robert @calm-artist-7206 and i think it's OK now https://github.com/OpenHands/agent-canvas/pull/945 maybe you can check and if its' OK merge or give feedback pls ty
āœ… 1
thanks on a stick 1