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

[마르시스에듀] 신규 하드웨어 로비코 추가 #782

Merged
merged 3 commits into from
Oct 23, 2024

Conversation

maru-shlee
Copy link
Contributor

유의사항

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

어떤 하드웨어 인가요?

[로비코 / 마르시스에듀]

어떤 종류의 PR 인가요?

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

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

  • 신규 하드웨어 로비코(RoBico) 추가

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

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

@Tnks2U
Copy link
Contributor

Tnks2U commented Oct 22, 2024

@maru-shlee

class Ozobot extends BaseModule {

js파일에서 클래스 선언시 기존에 있는 다른 회사의 클래스를 사용하고 있습니다.
이러면 타사의 클래스를 덮어씌우거나, 본인 파일이 덮어씌워질수도 있습니다.
BaseModule을 상속하되 타사와 겹치지 않는 독자적인 클래스명을 사용하시기 바랍니다.

@maru-shlee
Copy link
Contributor Author

@Tnks2U 잘못된 클래스 이름 수정하였습니다.

@Tnks2U
Copy link
Contributor

Tnks2U commented Oct 23, 2024

@maru-shlee
확인했습니다.
merge하겠습니다.

@Tnks2U Tnks2U merged commit f6241cc into entrylabs:develop-hw Oct 23, 2024
2 checks passed
@maru-shlee
Copy link
Contributor Author

@Tnks2U #776 코멘트에서 로비코의 샘플 제공 여부가 제공 예정으로 되어 있던데, 샘플 전달 받으신 것이 없는 건가요?

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