-
Notifications
You must be signed in to change notification settings - Fork 1
Add CI/testing pipelines + format code #24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
0bbc354 to
e7a2c68
Compare
|
The linter looks good! For the build_test workflow, does it actually test anything currently? I see that it calls docker compose build only, what is your intended way for a node to be tested? For the monorepo, I believe it calls the script inside https://github.com/WATonomous/wato_monorepo/tree/main/.github/templates/test . Either way, could you add some trivial tests to gym_vis and check if it runs? |
…_toolbox, added : jstest-gtk, added : display passthrough for (workso on linux), input passthough for devies into the container (gamepads can be used in the container now)
|
should be ready for review again! let me know if there's anything else to edit, or if I should squash the branch now |
Some pipelines that run on PRs against main to address issues #23 and #18. I also ran the auto formatter so it passes these tests.
See
(Tutorial) Micro Autonomy Testing Pipelines.mdin the changes for more details.auto-lintlabel to your PR to run the formatter