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

달고나 Plus 수정, 달고나 Basic 등록 #676

Merged
merged 2 commits into from
Oct 19, 2023

Conversation

dalgona-edu
Copy link
Contributor

@dalgona-edu dalgona-edu commented Oct 18, 2023

유의사항

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

어떤 하드웨어 인가요?

[달고나.에듀 Plus / 누구나(주)]

어떤 종류의 PR 인가요?

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

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

  • 풀업저항버튼 블록 기능 수정
  • 부저 on/off 블록 기능 수정

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

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

어떤 하드웨어 인가요?

[달고나.에듀 Basic/ 누구나(주)]

어떤 종류의 PR 인가요?

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

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

  • 신규 하드웨어 추가

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

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

@Tnks2U Tnks2U merged commit 89470c9 into entrylabs:develop-hw Oct 19, 2023
1 of 5 checks passed
@Tnks2U
Copy link
Contributor

Tnks2U commented Oct 19, 2023

@dalgona-edu
이번 PR에 buffer 관련 변수에 오타가 있어 테스트중 엔트리 하드웨어 프로그램이 멈추는 상황이 있었습니다. 수정해서 반영할 예정입니다. 유의 부탁드립니다.
수정사항 : b2a5068

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