https://docs.all-hands.dev/ logo
Join Slack
Powered by
`typescript-client` supports in CI Node 20 and 22....
# proj-agent-canvas
c
typescript-client
supports in CI Node 20 and 22. Does anyone know of reasons why we wouldn’t kill 20? I seem to recall we upgraded Local GUI a long time ago Thread in Slack Conversation
l
I am pretty sure this is vibe versioned and you can remove the 20
👍 1
c
@nutritious-balloon-65098 let's remove the obsolete node 20 from ci and anywhere. then clone openhands/ and agent-canvas/ from the same org and check what is it using; if it's 24 for both, let's make this 24 too. if not: open pr nondraft with the cleanup, and tell me wdyt , update and make them consistent
n
I'm on it! can track my progress here.
Since my last summary, no additional changes were made. Checklist: • ✅ Request addressed: Node 20 was removed from CI/tooling references. • ✅ Peer repos checked: 
OpenHands/OpenHands
and 
OpenHands/agent-canvas
were cloned and inspected. • ✅ Consistency decision applied: this repo now uses Node 22 as the baseline and adds Node 24 to the main CI matrix because the peer repos are not both on 24. • ✅ Validation completed: formatting, build, and unit tests passed. • ✅ Git workflow followed: changes were committed, pushed to a branch, and a non-draft PR was opened as requested. • ✅ Changes are concise: only CI workflow Node versions/setup-node versions and the contributing prerequisite were changed. No extraneous code changes were introduced. PR remains open here: https://github.com/OpenHands/typescript-client/pull/229
c
I think maybe this PR needs an admin merge, the Node 20 workflow was required https://github.com/OpenHands/typescript-client/pull/229
Open in Slack
PreviousNext
Powered by