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

Non lfs picknik_accessories #8

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Non lfs picknik_accessories #8

wants to merge 5 commits into from

Conversation

MikeWrock
Copy link
Collaborator

@MikeWrock MikeWrock commented Nov 15, 2024

Instead of telling users how git doesn't support subtrees and LFS, and how to work around it, we can just stop using LFS for this repo.

To test:

  • Delete moveit_pro_example_ws
  • Clone it
git clone -b non-lfs https://github.com/PickNikRobotics/moveit_pro_example_ws.git
  • Switch workspaces using moveit_pro configure
  • moviet_pro build
  • moveit_pro run -c lab_sim

@davetcoleman
Copy link
Member

I don't know enough about what picknik_accessories is, how big the file size is, and what the repercussion is of removing LFS. Can someone else review?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants