Closed shr-project closed 4 years ago
@tfoote please review this one -- you know quite a bit more than I do about this.
@herb-kuta-lge FYI
@herb-kuta-lge requested to split this pull request into smaller ones with separate issue tickets and to leave this original one for setup.py change he plans to update with different implementation based on @tfoote's feedback. I've created: https://github.com/ros-infrastructure/superflore/pull/265 https://github.com/ros-infrastructure/superflore/pull/266 https://github.com/ros-infrastructure/superflore/pull/267 https://github.com/ros-infrastructure/superflore/pull/268 https://github.com/ros-infrastructure/superflore/pull/269 https://github.com/ros-infrastructure/superflore/pull/270 to cover the other changes which were here yesterday.
@tfoote < Please review the latest implementation. I believe it addresses all of your concerns.
and small fix for superflore-datetime.inc.