All-Hands-AI / OpenHands

🙌 OpenHands: Code Less, Make More
https://all-hands.dev
MIT License
37.37k stars 4.23k forks source link

chore: Convert camelCase file names to kebab-case #5209

Closed amanape closed 14 hours ago

amanape commented 3 days ago

End-user friendly description of the problem this fixes or functionality that this introduces We have a pretty inconsistent file naming convention used throughout the frontend portion of the app.


Give a summary of what the PR does, explaining any non-trivial design decisions

Git CI

When working in environments like GitHub CI, file name conflicts arise because Git treats file names as case-insensitive (on case-insensitive file systems like macOS or Windows), while TypeScript treats file names as case-sensitive. This leads to errors when files with mismatched casings are resolved differently in the CI environment, especially on case-sensitive systems like Linux.

To address this, we need to clear both the TypeScript cache and Git cache during the build process.


Follow-up PRs


To run this PR locally, use the following command:

docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:326815d-nikolaik   --name openhands-app-326815d   docker.all-hands.dev/all-hands-ai/openhands:326815d
peltae commented 1 day ago

wait, it had a name all this time and it was kebab-case? lol