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

Enhance search function for not-loaded commits #2

Open
hansu opened this issue Jun 7, 2022 · 11 comments
Open

Enhance search function for not-loaded commits #2

hansu opened this issue Jun 7, 2022 · 11 comments

Comments

@hansu
Copy link
Owner

hansu commented Jun 7, 2022

The current search function searches only in commits that are already loaded.
So, search the complete history and load the regarding commit + "git-graph.repository.commits.loadMore" newer commits.
(It takes around 7 seconds on my system to load 40k commits but if such many commits are loaded, git-history reacts quite slow)
For reference in the commit message filter of https://github.com/vscode-ext-studio/vscode-git-graph is searches beyond the displayed commits.

@pktiuk
Copy link

pktiuk commented Jul 3, 2022

I see this feature issue is linked with mhutchie#607

@hansu
Are you going to somehow upstream your changes either upstream (although it seems abandoned for now) or to any other active fork?
Like for example https://github.com/vscode-ext-studio/vscode-git-graph

@hansu
Copy link
Owner Author

hansu commented Jul 3, 2022

I will create a PR to upstream as well, but I have no real hope that it will be merged in the near future.
But I will merge it here as one of the most active forks atm :). Then I will create a new release.
PS: check out my current release ;-)

@pktiuk
Copy link

pktiuk commented Jul 3, 2022

PS: check out my current release ;-)

For now I was just browsing commits and I found some good changes, but unluckily your version is nor listed in marketplace (even link in About section of your repo points at market site of legacy one)

@hansu
Copy link
Owner Author

hansu commented Jul 3, 2022

Yeah this is just meant as a tempory place to provide the changes as vsix file. I still have the hope that the official repo will become alive again. Otherwise I should think of publishing it on the market place, but I want to avoid the confusion to have three forks available there.

@pktiuk
Copy link

pktiuk commented Jul 3, 2022

You could also think about joining https://github.com/vscode-ext-studio, but it depends on their openness for other contributors (they have alreadypublished their version in marketplace, so it would save you some changes) :)

@hansu
Copy link
Owner Author

hansu commented Jul 3, 2022

Yes I know that, but they have done exactly zero for documenting their changes. I think that'sa no go when publishing. Furthermore they have changed the code structure so that the changes suitable for upstream cannot be applied. And they have disabled issues. These are the reasons why I wouldn't do that.

@pktiuk
Copy link

pktiuk commented Jul 3, 2022

That's a fair point.

@hansu
Copy link
Owner Author

hansu commented Jul 26, 2022

Yes I know that, but they have done exactly zero for documenting their changes. I think that's a no go when publishing.

I have to correct myself. Just two weeks before I wrote this, he adapted the documentation and also enabled issues.

@pktiuk
Copy link

pktiuk commented Jul 27, 2022

You can test their openness by opening PR with some fixes you have already done.

@hansu
Copy link
Owner Author

hansu commented Jul 27, 2022

Yeah but the changes on my repo (not done by me, thanks @LariWa) are based on the code of mhutchie - the code structure changes of cweijan would cause a lot of rework to make a suitable PR.

@hansu
Copy link
Owner Author

hansu commented Aug 4, 2022

[@pktiuk]

(even link in About section of your repo points at market site of legacy one)

I fixed that and the README as well ;-)

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

No branches or pull requests

2 participants