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

Fixed search for repository root directory #78

Merged
merged 1 commit into from Jan 27, 2024

Conversation

groegeorg
Copy link

In the situation, where one git/svn/hg repository is located inside the
directory tree of another, the search incorrectly selects the outer one
instead of the inner one.

In the situation, where one git/svn/hg repository is located inside the
directory tree of another, the search incorrectly selects the outer one
instead of the inner one.
@aaronliu0130 aaronliu0130 merged commit 3847a79 into cpplint:master Jan 27, 2024
@aaronliu0130
Copy link
Collaborator

Note that this wasn't merged correctly as the default branch is develop for some reason. We'll review the other PRs that merge into master and then get them into develop together.

@aaronliu0130
Copy link
Collaborator

aaronliu0130 commented Jan 29, 2024

Scratch that, now correctly committed in 8de9004.

@aaronliu0130
Copy link
Collaborator

@groegeorg It seems something about that commit credited your name but not your GitHub username. There are only two ways I know to rectify that: 1. Rebase to add you back and rewrite history, which may mangle commit dates. 2. You can maybe figure out some other contribution to make? 3. We can just let it be, as your name is still in the contribution list.

@aaronliu0130
Copy link
Collaborator

Actually, there's a much bigger authorship somewhere else, so we'll rebase

aaronliu0130 pushed a commit to aaronliu0130/cpplint that referenced this pull request Jan 29, 2024
In the situation, where one git/svn/hg repository is located inside the
directory tree of another, the search incorrectly selects the outer one
instead of the inner one.
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.

None yet

2 participants