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

Merge 2023 engineer rm control. #200

Closed
wants to merge 15 commits into from
Closed

Merge 2023 engineer rm control. #200

wants to merge 15 commits into from

Conversation

cc0h
Copy link
Contributor

@cc0h cc0h commented Oct 18, 2023

Merge 2023 engineer ui codes.Add can lose error in rm_hw.

@netlify
Copy link

netlify bot commented Oct 18, 2023

Deploy Preview for rm-control ready!

Name Link
🔨 Latest commit 423ad85
🔍 Latest deploy log https://app.netlify.com/sites/rm-control/deploys/654070a1ac5f5c00081c0881
😎 Deploy Preview https://deploy-preview-200--rm-control.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@@ -4,6 +4,7 @@

#include "rm_referee/ui/time_change_ui.h"

class ConstPtr;
Copy link
Contributor

Choose a reason for hiding this comment

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

为啥加这行?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

这行应该是merge的时候加进来的,没有注意到。

Copy link
Contributor

Choose a reason for hiding this comment

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

这行应该是merge的时候加进来的,没有注意到。

速速删除

@1moule
Copy link
Contributor

1moule commented Oct 19, 2023

pr标题跟描述可以改改,我看着就是在改ui,然后hw那个应该可以另开一个pr

@cc0h
Copy link
Contributor Author

cc0h commented Oct 19, 2023

pr标题跟描述可以改改,我看着就是在改ui,然后hw那个应该可以另开一个pr

hw只加了一句,就没想单独开一个pr,而且工程之前太多commit没pr,搞起来怪麻烦的

@cc0h cc0h changed the title Merge 2023 engineer Merge 2023 engineer rm control. Oct 19, 2023
@chenhuiYu00 chenhuiYu00 mentioned this pull request Oct 23, 2023
@cc0h cc0h closed this Oct 31, 2023
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.

4 participants