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

에이럭스 네모 오류 수정 #779

Merged
merged 2 commits into from
Oct 23, 2024
Merged

Conversation

hagwon
Copy link
Contributor

@hagwon hagwon commented Oct 16, 2024

유의사항

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

어떤 하드웨어 인가요?

[네모 / 에이럭스]

어떤 종류의 PR 인가요?

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

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

  • js 파일: 확장블록, 나침반 블록 값 오류 수정, 버전 체크 알람 추가
  • json 파일: 들여쓰기 수정

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

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

@Tnks2U
Copy link
Contributor

Tnks2U commented Oct 22, 2024

@hagwon
타깃 브랜치가 master로 되어 있습니다. develop-hw로 수정 바랍니다.

@hagwon hagwon changed the base branch from master to develop-hw October 22, 2024 07:05
@hagwon
Copy link
Contributor Author

hagwon commented Oct 22, 2024

변경하였습니다.

@Tnks2U Tnks2U merged commit 5ea3ae9 into entrylabs:develop-hw Oct 23, 2024
2 checks passed
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