Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Enable jrl-cmakemodules workspace compatiblity #225

Merged
merged 6 commits into from
Sep 6, 2024

Conversation

jorisv
Copy link
Contributor

@jorisv jorisv commented Jul 16, 2024

No description provided.

.gitmodules Outdated Show resolved Hide resolved
CMakeLists.txt Outdated Show resolved Hide resolved
unittest/CMakeLists.txt Show resolved Hide resolved
@jorisv jorisv force-pushed the topic/workspace branch from d0a7b4e to 336adea Compare July 24, 2024 08:34
@jorisv jorisv marked this pull request as ready for review July 24, 2024 08:42
@jorisv
Copy link
Contributor Author

jorisv commented Jul 24, 2024

@cmastalli Did the modifications are fine to you ?

@cmastalli
Copy link
Collaborator

@cmastalli Did the modifications are fine to you ?

I think we should not remove the in #225 (comment). However, I would like to hear from @nim65s on this topic.

@jorisv
Copy link
Contributor Author

jorisv commented Jul 26, 2024

@cmastalli I revert the change about add_project_dependency({eigenpy,pinocchio}).

If it's fine to you, you can merge.

@jcarpent
Copy link
Member

@cmastalli ping to merge this PR if you are fine.

@cmastalli
Copy link
Collaborator

@cmastalli ping to merge this PR if you are fine.

Hi @jcarpent -- We're just missing to address this comment: #225 (comment).

This should take no time to do it.

@cmastalli cmastalli merged commit abcd46f into Gepetto:devel Sep 6, 2024
2 checks passed
@jorisv jorisv deleted the topic/workspace branch September 6, 2024 09:57
nim65s added a commit to nim65s/robotpkg that referenced this pull request Dec 8, 2024
    ## [4.2.0] - 2024-12-08

    ### Added
    - Add PR2 robots ([210](Gepetto/example-robot-data#210))
    - Add GO2 robots ([239](Gepetto/example-robot-data#239))
    - Add Falcon Bravo7 Bluevolta robots ([244](Gepetto/example-robot-data#244))
    - Add alex robots ([247](Gepetto/example-robot-data#247))
    - Add compatibility with jrl-cmakemodules workspace ([225](Gepetto/example-robot-data#225))
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants