Skip to content

Update CODEOWNERS#217

Merged
chapulina merged 1 commit intoign-math6from
update_codeowners
Aug 12, 2021
Merged

Update CODEOWNERS#217
chapulina merged 1 commit intoign-math6from
update_codeowners

Conversation

@mjcarroll
Copy link
Copy Markdown

Follow-up from gazebosim/gz-physics#281

Signed-off-by: Michael Carroll michael@openrobotics.org

Signed-off-by: Michael Carroll <michael@openrobotics.org>
@mjcarroll mjcarroll requested a review from scpeters as a code owner August 2, 2021 19:23
@mjcarroll mjcarroll requested a review from chapulina August 2, 2021 19:23
@github-actions github-actions bot added 🏢 edifice Ignition Edifice 🏯 fortress Ignition Fortress 🏰 citadel Ignition Citadel 🔮 dome Ignition Dome labels Aug 2, 2021
@codecov
Copy link
Copy Markdown

codecov bot commented Aug 2, 2021

Codecov Report

Merging #217 (bc4f143) into ign-math6 (9519ce7) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           ign-math6     #217   +/-   ##
==========================================
  Coverage      99.21%   99.21%           
==========================================
  Files             65       65           
  Lines           6089     6089           
==========================================
  Hits            6041     6041           
  Misses            48       48           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9519ce7...bc4f143. Read the comment docs.

@chapulina chapulina enabled auto-merge (squash) August 2, 2021 20:49
@chapulina chapulina merged commit cf6d052 into ign-math6 Aug 12, 2021
@chapulina chapulina deleted the update_codeowners branch August 12, 2021 18:14
francocipollone pushed a commit that referenced this pull request Aug 17, 2021
Signed-off-by: Michael Carroll <michael@openrobotics.org>
scpeters added a commit that referenced this pull request Aug 17, 2021
* Basic setup for Python interface using SWIG
* Changes in cmake to run python tests with colcon
* Adding Vector3 and Vector4 python interface tests
* Changing from a single ign_math.i file to a ruby.i and a python.i

Signed-off-by: Marcos Wagner <wagnermarcos@ekumenlabs.com>

* Fixes install path and name math instead of pymath.
* Adds py examples
* Fixes ruby_TEST set up.
* Support both Swig3 and Swig4 versions.

Signed-off-by: Franco Cipollone <franco.c@ekumenlabs.com>

* Adds line in the src/CMakeLists.txt file to solve CI error.
Adds python test files to some of the classes that had an associated .i file.

Signed-off-by: LolaSegura <lsegura@ekumenlabs.com>

* Remove Marya from CODEOWNERS (#217)

Signed-off-by: Michael Carroll <michael@openrobotics.org>

* Use FAKE_INSTALL_PREFIX for PYTHONPATH in tests and
move the variable definition to the root CMakeLists.txt
* Set LD_LIBRARY_PATH for python tests
Point to fake install lib folder.

Signed-off-by: Steve Peters <scpeters@openrobotics.org>

Co-authored-by: Franco Cipollone <franco.c@ekumenlabs.com>
Co-authored-by: LolaSegura <lsegura@ekumenlabs.com>
Co-authored-by: Louise Poubel <louise@openrobotics.org>
Co-authored-by: Michael Carroll <michael@openrobotics.org>
Co-authored-by: Steve Peters <scpeters@openrobotics.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏰 citadel Ignition Citadel 🔮 dome Ignition Dome 🏢 edifice Ignition Edifice 🏯 fortress Ignition Fortress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants