RichShield FND Checked

[remoted]

부자실드에 대한 아날로그 및 디지털 센서에 대응하는 코드 작성 및 디버깅완료
부자실드에 대한 LCD에 대응하는 코드 작성 및 디버깅 완료
부자실드에 대한 FND에 대응하는 코드 작성 및 디버깅 완료

by Remoted 2020-12-31 02:37 AM

[wodnjs6512]

@remoted
변경점이 너무 많아 보이는데 develop-hw브랜치를 기준으로 작성하신 코드가 아니었던것으로 보입니다. 코어 로직쪽의 수정점도 보이는데, develop-hw를 작성하신 브랜치에 다시 한번 합치신후에 확인 부탁드립니다.

현재로써는 부자실드와 관련되지 않은 변경점이 너무 많습니다. 올려주신 PR에서 코어로직 쪽 변경점이 있다면 머지가 어렵습니다.

[wodnjs6512]

@remoted
앞으로의 PR은 아래의 PR형태와 같이 확인 후 올려주시길 부탁드립니다. 관련 코드 외의 다른 부분의 수정점이 보인다면 머지해드리기 어렵습니다.
기존 히스토리 확인해보니 마스터를 머지하신 기준으로 올려주셨는데, master 나 develop을 기준으로 올려주시면 확인이 어렵습니다.
#1970

[wodnjs6512]

@remoted
yarn.lock 안에서 코어로직 에서 사용되는 부분들이 제거 되어있습니다. 수정 바랍니다.
그리고 홍보용 이미지가 없습니다.
https://entrylabs.github.io/docs/guide/entryjs/2016-07-08-add_new_blocks4.html#%ED%95%98%EB%93%9C%EC%9B%A8%EC%96%B4-%EC%B2%AB-%EB%93%B1%EB%A1%9D%EC%8B%9C-%ED%99%8D%EB%B3%B4%EC%9A%A9-%EC%9D%B4%EB%AF%B8%EC%A7%80-%EC%B6%94%EA%B0%80

[remoted]

@wodnjs6512
22일 전 것도 수정해야 할까요? 아니면 해당 링크 관련된 수정만 진행하면 될지요?

[remoted]

@wodnjs6512
코어로직에서 사용하는 부분을 추가하는 방법을 알려주셨으면 합니다. _

[remoted]

@wodnjs6512
yarn.lock 파일관련 수정 완료했습니다.
이미지 추가가 완료되었습니다.

[wodnjs6512]

@remoted

아직 이슈가 남아있는듯 한데요 추가되어있던 패키지가 삭제되어있습니다.

@wodnjs6512
22일 전 것도 수정해야 할까요? 아니면 해당 링크 관련된 수정만 진행하면 될지요?

결과물에서 conflict나 문제점이 없으면 됩니다.

@wodnjs6512
코어로직에서 사용하는 부분을 추가하는 방법을 알려주셨으면 합니다. _

코어 로직을 수정하시려는 이유가 궁금한데요 어떠한 부분으로 사용하시려는 것인지 확인이 필요합니다.
그리고 src/class/DataTable.js 부분이 변경 되어있는데요. 여기를 변경하기를 원하시는걸까요?

[remoted]

@wodnjs6512
일단 제 경우에는 추가되어있던 패키지를 제거하거나 그런건 없구요
yarn.lock 파일에서 상이점이 있다고 하셔서 git fetch upstream/master 를 이용하여 fetch 이후에 다시 가져오긴 했습니다.

코어 로직을 따로 수정한 것이 아니라, git fetch 로 반영을 시켰는데요..
따로 코어로직을 변경시키려는 것은 없습니다. _

[wodnjs6512]

@remoted
저희쪽에서 드리는 가이드에 의하면 master가 아닌 develop-hw를 기준으로 진행하시기를 요청 드리고 있는데요
https://entrylabs.github.io/docs/guide/entryjs/2018-03-09-getting_started.html#%ED%81%B4%EB%A1%A0

develop-hw 기준으로 rebase한번 진행하여서 PR수정해주시면 될것 같습니다. entryjs쪽에도 같은 방식으로 진행을 하신것으로 보이기에, 같은 방식으로 진행하여 주시면 될것 같습니다.

[remoted]

@wodnjs6512
곧 확인해서 PR올리겠습니다. _

[remoted]

@wodnjs6512
변경점 확인 부탁드립니다. _

기존에도 develop-hw 에서 작성하고 있었는데요!

$ git checkout master
$ git fetch origin master
$ git rebase develop-hw

로 진행한 이후에
다시 commit 과 push 진행하였는데 제대로 한게 맞을까요?

[wodnjs6512]

@remoted
사용하고 계신 develop-hw 브랜치 내에서

$ git rebase upstream/develop-hw

를 진행하신후에 필요한 부분만 다시 커밋 하셔서 올려주시면 될것 같습니다.

[remoted]

@wodnjs6512
RichShield js 및 hw 파일 rebase 완료했습니다.

[wodnjs6512]

@remoted
rebase진행을 어떻게 하신지는 모르겠으나, 해당 부분들이 전혀 수정 되지 않은 것으로 보이는데요.....

@remoted

아직 이슈가 남아있는듯 한데요 추가되어있던 패키지가 삭제되어있습니다.

@wodnjs6512
22일 전 것도 수정해야 할까요? 아니면 해당 링크 관련된 수정만 진행하면 될지요?

결과물에서 conflict나 문제점이 없으면 됩니다.

@wodnjs6512
코어로직에서 사용하는 부분을 추가하는 방법을 알려주셨으면 합니다. _

코어 로직을 수정하시려는 이유가 궁금한데요 어떠한 부분으로 사용하시려는 것인지 확인이 필요합니다.
그리고 src/class/DataTable.js 부분이 변경 되어있는데요. 여기를 변경하기를 원하시는걸까요?

이 부분들이 전혀 바뀌어 있지 않습니다....
만약 git에 익숙하지 않으신거라면, github desktop과 같은 프로그램을 사용하는것을 추천드립니다...

그리고, rebase를 진행 하게 된다면 rebase 기준브랜치와 비교하여서 다른점들이 로컬 커밋 상태로 들어오는데, 그 로컬 커밋들을 전부 revert하신후, 필요없는 부분의 변경점을 discard해주신후에 필요한 변경점만 commit을 해서 올려달라는 요청이었습니다.

[wodnjs6512]

@remoted
일단 너무 오랜기간 동안 묶여있게 되어서 제가 임의로 코드 rebase 수정 도와드렸습니다.
앞으로의 경우 PR관련하여서 같은 이슈로 도와드릴 수 없으니, 확인 부탁드립니다.
지금 현재 rebase한 상태를 확인 하신후 PR reopen부탁드립니다.

[remoted]

@wodnjs6512
너무 감사드립니다 ㅠㅠ...

[wodnjs6512]

@remoted
현재상태의 문제가 없으시다면 지금은 PR들이 close상태로 되어있으니 entry-hw와 entryjs PR들을 전부 reopen해 주시기 바랍니다.