Skip to content

Update maintainers across ROS packages - #70

Merged
woensug-choi merged 2 commits into
IOES-Lab:ros2from
yeseorizi:chore/update-maintainers-20260923
Sep 23, 2026
Merged

woensug-choi merged 2 commits into
IOES-Lab:ros2from
yeseorizi:chore/update-maintainers-20260923

Conversation

@yeseorizi

Copy link
Copy Markdown
Collaborator

Summary

  • add Woen-Sug Choi (wschoi@kmou.ac.kr) as a maintainer to all 13 ROS packages
  • add Ye-Seol Kwon (kys50728@g.kmou.ac.kr) as a maintainer to all 13 ROS packages
  • preserve the existing package maintainers and normalize the existing Woen-Sug Choi entries to the institutional address

Validation

  • parsed all 13 package.xml files as XML
  • confirmed exactly one entry for each newly standardized maintainer in every package
  • git diff --check passed

This PR only updates package metadata and does not change runtime behavior.

@yeseorizi

Copy link
Copy Markdown
Collaborator Author

The GitHub lint failure is unrelated to the package.xml changes: the hosted runner used a stale apt index and received HTTP 404 while installing libxml2-utils. PR #71 refreshes apt indexes before system dependency installation and its lint workflow passes. The package metadata itself also passes pre-commit.ci and all 13 manifests parse successfully as XML.

@yeseorizi

Copy link
Copy Markdown
Collaborator Author

Manifest and integration validation update

  • All 13 package.xml files parse successfully as XML.
  • Each manifest contains exactly one institutional entry for Woen-Sug Choi <wschoi@kmou.ac.kr> and one for Ye-Seol Kwon <kys50728@g.kmou.ac.kr>.
  • Existing third-party maintainers were preserved.
  • This PR was included in a local pre-merge candidate with Fix BlueROV IMU bridge for ArduSub SITL #66–69 and the apt-index fix in Refresh apt indexes before CI and Docker dependency installs #71. The resulting native ARM64 Docker image built successfully, and all 14 executable public Quickstarts passed with graceful shutdown.

The red GitHub lint check on this PR is the stale apt-index 404 already addressed by #71; pre-commit.ci passed and the manifest contents were revalidated locally.

@woensug-choi
woensug-choi merged commit aa184c4 into IOES-Lab:ros2 Sep 23, 2026
3 of 5 checks passed
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