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

Update uctags.vim #887

Merged
merged 2 commits into from
Jul 24, 2024
Merged

Update uctags.vim #887

merged 2 commits into from
Jul 24, 2024

Conversation

sqlmap3
Copy link
Contributor

@sqlmap3 sqlmap3 commented Jul 24, 2024

@alerque
Copy link
Member

alerque commented Jul 24, 2024

Please edit the file(s) in the branch from the original PR #885 instead of opening new PRs. It's much more difficult to review and track feedback when you spread thet same contribution out across multiple PRs for each adjustment.

@alerque alerque closed this Jul 24, 2024
@alerque
Copy link
Member

alerque commented Jul 24, 2024

Oh yuck you deleted the original PRs branch, so GH can't even help untangle this.

@alerque alerque reopened this Jul 24, 2024
Incorporating review comments as per note from submitter
@raven42 raven42 merged commit 1690b19 into preservim:master Jul 24, 2024
5 checks passed
@raven42
Copy link
Collaborator

raven42 commented Jul 24, 2024

Thanks for the contribution. The PR has been merged in with the suggested changed.

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.

3 participants