Skip to content

Fixed issue #2188 #3091

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

Fixed issue #2188 #3091

wants to merge 2 commits into from

Conversation

KishkinJ10
Copy link
Contributor

No description provided.

@PRMerger6 PRMerger6 requested a review from colin-home April 16, 2021 13:03
@PRMerger6
Copy link
Contributor

@KishkinJ10 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver ktoliver added the aq-pr-triaged Tracking label for the PR review team label Apr 16, 2021
@colin-home
Copy link
Contributor

@KishkinJ10
I'm sorry, it looks like this issue was already fixed in PR #2991 last July, but the original issue wasn't closed because it wasn't linked in that PR. This fix isn't required, so I'm closing it, but I appreciate the effort that went into it.

@colin-home colin-home closed this Apr 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@colin-home colin-home Awaiting requested review from colin-home

Assignees

@colin-home colin-home

Labels
aq-pr-triaged Tracking label for the PR review team Change sent to author cpp-cx/tech do-not-merge visual-cpp/prod
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@KishkinJ10 @PRMerger6 @colin-home @ktoliver