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

[vscode] git 연결된 레포지토리가 확장 개발 호스트에서 실행되지 않는 버그 임시 수정 #730

Merged
merged 1 commit into from
Sep 27, 2024

Conversation

seungineer
Copy link
Member

Related issue

  • '확장개발호스트'에서 githru가 열리지 않는 버그
  • 레포지토리 내에 git이 연결되어 있음에도 panel이 dispose됩니다.

Result

Work list

  • 레포지토리의 git 연결 여부를 확인하던 부분을 주석 처리하였습니다.
  • analyzedData 값의 할당 유무로 git 연결 여부를 확인하였으나 git 연결된 경우에도 analyzedData에는 undefined가 할당됩니다.

Discussion

  • analyzedData에 항상 undefined 값이 할당되는 원인을 모르겠습니다. 🥲

@seungineer seungineer added bug Something isn't working 💻 vscode labels Sep 21, 2024
@seungineer seungineer added this to the v0.7.2 milestone Sep 21, 2024
@seungineer seungineer requested review from a team as code owners September 21, 2024 14:15
Comment on lines +90 to +94
//FIXME - For repositories where git exists, analyzedData is initially assigned as undefined.
// if (!analyzedData) {
// this.dispose();
// throw new Error("Project not connected to Git.");
// }
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

음. 이 부분 @DaYoung-woo 님이 #679 에서 수정해주신 부분 같은데
한번 같의 논의해보시면 좋을 것 같습니다!

혹시 지금 아예 동작이 안되는 상황이라면 일단 바로 반영해주시구요!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아예 확장개발호스트가 열리지 않는 상황이라
해당 내용 issue로 생성 후 해당 PR은 merge 하도록 하겠습니다.

Copy link
Contributor

@lxxmnmn lxxmnmn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@seungineer seungineer merged commit ee60dac into githru:main Sep 27, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working 💻 vscode
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants