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

Develop hw #647

Merged
merged 2 commits into from
Jul 20, 2023
Merged

Develop hw #647

merged 2 commits into from
Jul 20, 2023

Conversation

irarykim
Copy link
Contributor

@irarykim irarykim commented Jul 18, 2023

유의사항

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

어떤 하드웨어 인가요?

[실과로봇 / 로보티즈]

어떤 종류의 PR 인가요?

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

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

  • 제어기 연결 후 펌웨어 업데이트 버튼 추가
  • 제어기 펌웨어가 구버전이면 업데이트 안내창 표시

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

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

@Tnks2U
Copy link
Contributor

Tnks2U commented Jul 20, 2023

@irarykim
안녕하세요.
펌웨어 관련 수정 PR 감사합니다.
7월은 일전에 공지드린대로 서버배포는 없습니다만, entry-hw 설치 프로그램은 배포가 될 예정입니다.
이 변경사항은 7월에 배포 희망하시는 걸까요?

@irarykim
Copy link
Contributor Author

@irarykim 안녕하세요. 펌웨어 관련 수정 PR 감사합니다. 7월은 일전에 공지드린대로 서버배포는 없습니다만, entry-hw 설치 프로그램은 배포가 될 예정입니다. 이 변경사항은 7월에 배포 희망하시는 걸까요?

안녕하세요,

네 그렇습니다.

감사합니다.

@Tnks2U
Copy link
Contributor

Tnks2U commented Jul 20, 2023

@irarykim
네 알겠습니다.
변경된 기능 테스트는 로보티즈IoT(실과로봇과 동일하다 들었습니다.)로 진행하겠습니다.
코드 검토 & 일부 리팩토링이 진행될 수 있습니다.

@Tnks2U Tnks2U merged commit 818f2fb into entrylabs:develop-hw Jul 20, 2023
2 checks passed
@Tnks2U Tnks2U mentioned this pull request Jul 26, 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.

2 participants