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

Referring repos with their git-dir instead of their work-tree. #614

Merged
merged 7 commits into from
Nov 20, 2023

Conversation

ysalmon
Copy link
Contributor

@ysalmon ysalmon commented Aug 7, 2023

Also sets up display routines to avoid displaying a repo as ".git" : we display the penultimate component of the path when the last one is ".git". This assumes nobody creates a repo in /.git.

This closes #612.

This needs review, especially wrt opening "remote" repos, which I did not test.

One will need to purge one's repo history on upgrading to this version.

@Murmele
Copy link
Owner

Murmele commented Aug 27, 2023

This needs review, especially wrt opening "remote" repos, which I did not test.

What do you mean here?

@ysalmon
Copy link
Contributor Author

ysalmon commented Aug 27, 2023

Gittyup has an interface to connect to repo from Github, GitLab, etc. I did not check if my patch affects this.

@Murmele
Copy link
Owner

Murmele commented Aug 31, 2023

I will check it out as next. Sorry for having to wait. Quite busy at the moment

@ysalmon
Copy link
Contributor Author

ysalmon commented Aug 31, 2023

Not a problem ! I am not waiting btw, I'm using my fork right now, so I already enjoy the feature I was missing.

@Murmele
Copy link
Owner

Murmele commented Nov 12, 2023

Old behaviour: Using the workdir path

image

…ectly when requesting the path, otherwise the settings are stored inconsistently and might lead to an infinity loop in void RecentRepositories::load()
@Murmele
Copy link
Owner

Murmele commented Nov 12, 2023

@ysalmon for me this part with .git makes problems and I am entering an infinity loop.
Can you check my changes and retest?

@Murmele Murmele merged commit 69d3e21 into Murmele:master Nov 20, 2023
11 checks passed
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.

Gittyup gets confused with work-tree out of git-dir
2 participants