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

Renaming develop branch to ros1 #577

Closed
jihoonl opened this issue Jul 29, 2021 · 7 comments
Closed

Renaming develop branch to ros1 #577

jihoonl opened this issue Jul 29, 2021 · 7 comments

Comments

@jihoonl
Copy link
Member

jihoonl commented Jul 29, 2021

I plan to rename develop branch as ros1 to slowly move toward clear visibility for both ros1 and ros2 supports. Please let me know if anyone has concerns about this branch renaming. cc: @RobotWebTools/active @RobotWebTools/authors

With no concern raises, I will rename develop to ros1 on the first week of August.

@amacneil
Copy link
Member

I support this

@flynneva
Copy link
Contributor

with this change will the default branch switch then to the new name or will you also switch the default branch? I've noticed a lot of core ros packages switching their default branch to the ROS2 develop or rolling branch.

@amacneil
Copy link
Member

amacneil commented Jul 29, 2021

Related: is there any consistency in ROS packages between calling the default branch main / master / develop? I notice a lot of RobotWebTools packages use develop, but there doesn't seem to be consistency between other ROS packages I reviewed.

I would advocate for just using main for the ros2 branch since it is the new github default (which e.g. nav2 uses).

@flynneva
Copy link
Contributor

I would advocate for just using main for the ros2 branch

seconded!

@jihoonl
Copy link
Member Author

jihoonl commented Aug 10, 2021

Sorry for the late responses guys.

@flynneva

with this change will the default branch switch then to the new name or will you also switch the default branch? I've noticed a lot of core ros packages switching their default branch to the ROS2 develop or rolling branch.

the default branch will become ros2 at some point. But, I would prefer to do it step by step with a slower transition as there are more ros1 users than ros2 as far as I know at this moment. I will definitely change the default branch to ros2 if I start to see more active development happens in ros2.

@amacneil

Related: is there any consistency in ROS packages between calling the default branch main / master / develop? I notice a lot of RobotWebTools packages use develop, but there doesn't seem to be consistency between other ROS packages I reviewed.

It is due to the historic reason as Robot Web Tool has been 7-8 years around. The default branch of github and ros at that time was master and develop. Thus, it was just not changing since then. 😄

I would advocate for just using main for the ros2 branch since it is the new github default (which e.g. nav2 uses).

I would have used main for ros2 if rosbridge is solely for ros2. However, I think using ros1 and ros2 separately more makes sense at this moment as the repo supports both releases.

I believe nav2 uses main because it is designed for ros2 since its start. If any repo is starting solely for ros2, I definitely agree that using main for ros2 support is clear.

How do you think?

@amacneil
Copy link
Member

Makes sense. I think calling the branches ros1 and ros2 is fine.

I think it would also be ok to call the ros2 branch main, but maybe calling it ros2 is more explicit.

@amacneil
Copy link
Member

I have renamed develop to ros1 and updated the default branch to ros2.

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

No branches or pull requests

3 participants