Skip to content

Change David's Email Address#5388

Merged
SteveMacenski merged 1 commit intoros-navigation:mainfrom
DLu:maintainer_email
Jul 24, 2025
Merged

Change David's Email Address#5388
SteveMacenski merged 1 commit intoros-navigation:mainfrom
DLu:maintainer_email

Conversation

@DLu
Copy link
Contributor

@DLu DLu commented Jul 24, 2025

Description of contribution in a few bullet points

Just changing where buildfarm emails go.

For Maintainers:

  • Check that any new parameters added are updated in docs.nav2.org
  • Check that any significant change is added to the migration guide
  • Check that any new features OR changes to existing behaviors are reflected in the tuning guide
  • Check that any new functions have Doxygen added
  • Check that any new features have test coverage
  • Check that any new plugins is added to the plugins page
  • If BT Node, Additionally: add to BT's XML index of nodes for groot, BT package's readme table, and BT library lists
  • Should this be backported to current distributions? If so, tag with backport-*.

Signed-off-by: David V. Lu <davidvlu@gmail.com>
@DLu DLu force-pushed the maintainer_email branch from b2b3d62 to 2812800 Compare July 24, 2025 17:02
@SteveMacenski
Copy link
Member

SteveMacenski commented Jul 24, 2025

No objections - but do you want emails from Nav2? We didn't mean to put you on the hook for supporting this in ROS 2 by originally pulling this in from robot_navigation. We could remove your email and put mine in place. But up to you!

@DLu
Copy link
Contributor Author

DLu commented Jul 24, 2025

I am happy to be listed as the maintainer still, and wouldn't object to being put in CODEOWNERS for the set of packages, but at this time I have very little invested in the state of the packages from the buildfarm.

@SteveMacenski SteveMacenski merged commit 6af5c23 into ros-navigation:main Jul 24, 2025
14 checks passed
SakshayMahna pushed a commit to SakshayMahna/navigation2 that referenced this pull request Aug 8, 2025
Signed-off-by: David V. Lu <davidvlu@gmail.com>
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