Skip to content

Conversation

@jdlee0320
Copy link
Contributor

No description provided.

@Tnks2U
Copy link
Collaborator

Tnks2U commented Jul 20, 2023

@jdlee0320
file changed를 보시면 block_analysis.js
block_analysis.js 파일을 변경하셨는데, 이유가 있으실까요? 공용파일은 기본적으로 변경이 불가합니다.

@jdlee0320
Copy link
Contributor Author

jdlee0320 commented Jul 20, 2023 via email

@Tnks2U
Copy link
Collaborator

Tnks2U commented Jul 20, 2023

@jdlee0320
file changed에 포함되어 있을경우, 무시하고 반영하면 해당 파일이 변경되게 됩니다.
일단 제가 임시로 해당 파일 변경사항을 없애는 커밋을 추가해 두었습니다.

추후에 변경사항이 있을 경우엔 불필요한 changed 사항이 없는지 체크를 부탁드립니다.

@jdlee0320
Copy link
Contributor Author

jdlee0320 commented Jul 20, 2023 via email

@Tnks2U
Copy link
Collaborator

Tnks2U commented Jul 20, 2023

@jdlee0320
넵, PR 상으로는 문제가 없어 보여서, merge 후 체크하겠습니다.
실제 테스트는 사측 샘플이 도착하면 진행될 예정입니다.
신규 하드웨어여서 entryjs쪽 수정이 있는 관계로 배포는 8월에 진행될 예정입니다.

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.

3 participants