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

[git-scm] No repository found #7860

Closed
502647092 opened this issue May 21, 2020 · 5 comments · Fixed by #7870
Closed

[git-scm] No repository found #7860

502647092 opened this issue May 21, 2020 · 5 comments · Fixed by #7870
Assignees
Labels
git issues related to git

Comments

@502647092
Copy link
Contributor

Bug Description:

image

  • I manually add paths to local storage theia:/:file:///home/project:theia-git-all-repositories
  • refresh ide Can fix it
    image
    image

Steps to Reproduce:

  1. build latest theia
  2. open a WorkSpace have multi git repo

Additional Information

  • Operating System: Windows10
  • Theia Version: 1.2.0-next.f97ec33e
@vince-fugnitto
Copy link
Member

Duplicate of #4107?
You can try using the git and git-ui builtin extensions and determining if the use-case is handled correctly (@theia/git will be deprecated in the near future).

@vince-fugnitto vince-fugnitto added the git issues related to git label May 21, 2020
@502647092
Copy link
Contributor Author

@vince-fugnitto
image
also have this problem

@westbury
Copy link
Contributor

westbury commented May 22, 2020

This is a regression caused by ecdb151 . With the typescript upgrade, the locator is no longer returning the repositories.

The problem is caused by the change of this line:

if (stat.isDirectory) {
to
if (stat.isDirectory()) {

The original line was always true, so the code inside the 'if' was executed for files. Executing the code only for directories results in repositories not being returned.

@502647092
Copy link
Contributor Author

another problem if ide reconnect backend the git info will lose

@westbury
Copy link
Contributor

@502647092 I have pushed a fix to branch fix-locate-repos. Can you test with this branch to see if it fixes the issues?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
git issues related to git
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants