search

hello-robot / stretch_web_teleop

Remote web teleoperation for the Stretch mobile manipulators from Hello Robot Inc.
Other
13 stars 0 forks source link

WIP: Add JS(X)/TS(X) linting #45

Open hello-amal opened 3 months ago

hello-amal commented 3 months ago

Description

This PR, in partial service of #43 , builds off of #44 to add JS(X)/TS(X) linting.

This PR is a work-in-progress; although the pre-commit hooks are properly configured to lint the repository (and that has been tested), they generate hundreds of warnings/errors. Most are straightforward to fix, but some fixes require legacy knowledge of the codebase (e.g., the fact that relativePose no longer exists as a type). Those lint errors are better suited for @hello-vinitha to address.

Testing procedure

TODO

Before opening a pull request

From the top-level of this repository, run:

To merge

hello-amal commented 3 months ago

At this point, CI is (rightly) failing due to the linting errors. But CI configuration is correct.