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

AIBOT 파일명, 접속 방식 변경, ToyBot 생성 #704

Merged
merged 4 commits into from
Jan 18, 2024
Merged

AIBOT 파일명, 접속 방식 변경, ToyBot 생성 #704

merged 4 commits into from
Jan 18, 2024

Conversation

hagwon
Copy link
Contributor

@hagwon hagwon commented Jan 18, 2024

유의사항

  • base Branch 는 develop-hw 로 통일하였습니다. master, develop 이 아닙니다.
  • 체크박스의 체크는 이슈 생성 후 직접 클릭 or [x] 와 같이 체크하면 됩니다.

어떤 하드웨어 인가요?

[AIBOT / (주)지니로봇]
[ToyBot / (주)지니로봇]

어떤 종류의 PR 인가요?

  • 신규 하드웨어 추가
  • 기존 하드웨어 변경
  • 기타

이 PR 에서 어떤 부분이 변경되었나요? (작성필요)

  • AIBOT 파일명 변경

  • app/modules/aibot.js -> app/modules/jinirobot_aibot.js

  • app/modules/aibot.json -> app/modules/jinirobot_aibot.json

  • app/modules/aibot.png -> app/modules/jinirobot_aibot.png

  • AIBOT 접속 시 컴포트 선택 창 false 설정

  • ToyBot 생성

PR 에 대해 아래 부분을 확인해주세요

  • 코드 문법 오류가 발생하진 않았는가
  • 코딩 컨벤션을 정리했는가 (린트, 들여쓰기, 등등..)
  • base branch 설정을 develop-hw 으로 했는가?
  • PR 생성 후, 기존 코드와 충돌이 나진 않았나요?

@Tnks2U
Copy link
Contributor

Tnks2U commented Jan 18, 2024

@hagwon
다음부턴, 기기별로 PR을 따로 해주시길 부탁드립니다.
기기를 2종류 이상 한번에 PR하시면 한쪽이 문제가 생겨서 롤백을 할때, 나머지 기기들도 함께 롤백이 됩니다.

@Tnks2U Tnks2U merged commit 26d11fa into entrylabs:develop-hw Jan 18, 2024
2 checks passed
@hagwon
Copy link
Contributor Author

hagwon commented Jan 18, 2024

네 알겠습니다. 다음부턴 주의하겠습니다. 감사합니다.

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.

2 participants