Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

feat: add initial CHANGELOG.rst for all packages #207

Conversation

sasakisasaki
Copy link

Description

Adds CHANGELOG.rst for all packages in this repository.

How was this PR tested?

As this PR is not a fix for software's functionality of the packages, I did not perform any software testing. I only checked if there is no any unrelated file in this PR.

Notes for reviewers

Any requests are appreciated. For example, I'm welcome to do the requested checks and so on.

Effects on system behavior

As this PR is merged, you'll need to manage all the CHANGELOG.rsts in every version update. Perhaps this will be kind of additional tasks for users in this repository.

Copy link

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

Copy link
Contributor

@isamu-takagi isamu-takagi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM (rqt_diagnostic_graph_monitor)

Copy link
Member

@youtalk youtalk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please run catkin_prepare_release too.

@sasakisasaki
Copy link
Author

sasakisasaki commented Jan 28, 2025

@youtalk Thank you for providing your comment. I tried the command catkin_prepare_release and observed the following error. Do I need to fix the version so all the packages have the same version together?

(EDITED) I understand that all the packages in this repository must have the consistent version.

$ catkin_prepare_release
Prepare the source repository for a release.
Repository type: git
Found packages: rqt_diagnostic_graph_monitor, bag2lanelet, tier4_logging_level_configure_rviz_plugin, autoware_sample_vehicle_description, tier4_simulated_clock_rviz_plugin, autoware_debug_tools, tier4_calibration_rviz_plugin, tier4_screen_capture_rviz_plugin, tier4_automatic_goal_rviz_plugin, tier4_target_object_type_rviz_plugin, mission_planner_rviz_plugin, tier4_calibration_msgs, tier4_debug_tools, tier4_debug_rviz_plugin, tier4_control_rviz_plugin, rtc_manager_rviz_plugin, tier4_string_viewer_rviz_plugin, autoware_dependency_checker, simulator_compatibility_test, control_data_collecting_tool, driving_environment_analyzer, estimator_utils, time_delay_estimator, pitch_checker, calibration_adapter, parameter_estimator, autoware_lanelet2_map_utils, autoware_pointcloud_divider, autoware_pointcloud_merger, autoware_lanelet2_map_validator, deviation_evaluator, deviation_estimator, autoware_planning_data_analyzer, autoware_static_centerline_generator, planning_debug_tools, autoware_route_client, autoware_rtc_replayer, vehicle_cmd_analyzer, control_debug_tools, tier4_metrics_rviz_plugin
Two packages have different version numbers (0.0.0 != 0.1.0):
- ./bag2lanelet/package.xml
- ./system/rqt_diagnostic_graph_monitor/package.xml

@youtalk
Copy link
Member

youtalk commented Jan 28, 2025

@sasakisasaki Instead of explaining, it would be faster for me to do it myself and have you learn from the commit history, so I’ll handle it that way this time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants