Skip to content
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

Cleanup meta scripts and add .vscode #198

Merged
merged 16 commits into from
Aug 31, 2023
Merged

Cleanup meta scripts and add .vscode #198

merged 16 commits into from
Aug 31, 2023

Conversation

jaagut
Copy link
Member

@jaagut jaagut commented Aug 3, 2023

Proposed changes

  • Split requirements.txt in
    1. requirements/common.txt,
    2. requirements/robot.txt and
    3. requirements/dev.txt
      • Add black, pre-commit and ruff deps
  • Add .vscode config for extensions and word list for spelling
  • Refactor Makefile
    • Create scripts and make commands to set up different tools:
      • basler: Same as before, but separated in own script to be callable from multiple places inside the Makefile
      • pip: `Install and upgrade pip requirements for a development environment
      • pre-commit: Try to install pre-commit hooks for every submodule
      • rosdep: Update and install rosdep dependencies
      • vscode: Install recommended VS Code extensions
    • Create commands that bundle previous commands: install and update
    • Remove deprecated commands doc and install.pl
  • Removes ROS 2 migration tool catkin-to-ament.py

Related issues

bit-bots/bitbots_behavior/pull/331
bit-bots/wolfgang_robot/pull/279
bit-bots/bitbots_tools/pull/153
https://git.mafiasi.de/Bit-Bots/ansible/pulls/52

@jaagut jaagut requested review from texhnolyze and timonegk August 3, 2023 16:55
@jaagut
Copy link
Member Author

jaagut commented Aug 3, 2023

@jaagut jaagut changed the title Update dependencies and add vscode extensions docs Cleanup meta scripts and add .vscode Aug 11, 2023
@jaagut jaagut marked this pull request as ready for review August 11, 2023 22:15
@jaagut jaagut requested a review from Flova August 11, 2023 22:20
Flova
Flova previously approved these changes Aug 12, 2023
Makefile Show resolved Hide resolved
@jaagut jaagut merged commit c3333ce into master Aug 31, 2023
@jaagut jaagut deleted the feature/update_deps branch August 31, 2023 17:26
Flova added a commit that referenced this pull request Jan 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants