Got inspired from <@U070UP4FW3F> difficulty settin...
# proj-agent-canvas
g
Got inspired from @elegant-branch-54423 difficulty setting up the dev environment when testing someone's branch So, I have updated our
development.md
OH has already done 2 passes; lmk if those 2 minor things are important enough to fix otherwise I think it's good. I did test it; see comments and screenshots. PR is ready for review: https://github.com/OpenHands/OpenHands/pull/13432 cc: @better-park-34863
✅ 1
I think it's not letting me merge the PR because I lack the permissions. it just gives me the option to keep "update branch" even though i approved auto-merge If you have time either @calm-artist-7206 or @better-park-34863 could you please merge this? Or, happy to take feedback if I am doing something wrong. There is now a check failing but OH says its a rate limit issue. It was not failing previously.
b
It's still running CI tests
🆗 1
c
I'll take care of that, just made a few small tweaks to remove V0 stuff from sight, so that it doesn’t confuse humans and agent alike 😅 I left a tiny comment on the same thing
👀 1
ty 1
If you want, we can merge as is, and maybe consider that remaining section in a follow up?
g
to confirm, this remaining section
There is no more headless or CLI here, in this repository, in V1. The remaining code is legacy V0, which is often fooling agents and humans that it’s still applicable.
IMHO we would be better to remove all references to these from these docs, WDYT?
c
Yes, where it says CLI or headless and gives instruction on that
👀 1
Those don’t exist anymore in this repo, for V1
From “`If you are running headless or CLI workflows, you can prepare local defaults with:”` and what it says after
We merge, so it doesn’t run CI anymore? And then you can have OH figure out that
👍 1
Done
g
Sounds good thank you all for the help i can start kicking off cleaning up v0 instructions
ty 1