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

Add IR sensor reset function #777

Merged
merged 2 commits into from
Oct 24, 2024
Merged

Conversation

myunghwahong
Copy link

유의사항

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

어떤 하드웨어 인가요?

[알티노 / (주)새온]

어떤 종류의 PR 인가요?

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

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

  • 알티노 IR 센서 초기화 블록 추가

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

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

@Tnks2U
Copy link
Contributor

Tnks2U commented Oct 22, 2024

@myunghwahong
구체적으로 어떤 부분을 테스트해야 하는지 기입해주셔야 테스트가 가능합니다.
실물 테스트를 희망하신다면, 테스터분이 이해할 수 있게 구체적인 변경사항 기입을 부탁드립니다.

@Tnks2U
Copy link
Contributor

Tnks2U commented Oct 23, 2024

@myunghwahong
js 파일 최상단에 스쿨키트의 'motor' 모듈을 불러오는 코드가 있습니다.
타사의 모듈이고 사용하고 계시지도 않아서 실수로 보이는데요.
제거 부탁드립니다.

@Tnks2U Tnks2U merged commit cf8d802 into entrylabs:develop-hw Oct 24, 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