-
Notifications
You must be signed in to change notification settings - Fork 89
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
Merge 2023 engineer rm control. #200
Conversation
# Conflicts: # rm_msgs/CMakeLists.txt # rm_referee/src/referee_base.cpp
✅ Deploy Preview for rm-control ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
rm_referee/src/ui/time_change_ui.cpp
Outdated
@@ -4,6 +4,7 @@ | |||
|
|||
#include "rm_referee/ui/time_change_ui.h" | |||
|
|||
class ConstPtr; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
为啥加这行?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这行应该是merge的时候加进来的,没有注意到。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这行应该是merge的时候加进来的,没有注意到。
速速删除
pr标题跟描述可以改改,我看着就是在改ui,然后hw那个应该可以另开一个pr |
hw只加了一句,就没想单独开一个pr,而且工程之前太多commit没pr,搞起来怪麻烦的 |
Merge 2023 engineer ui codes.Add can lose error in rm_hw.