<optional>: Remove stale LWG-3627 comments by tycket033-tech · Pull Request #6416 · microsoft/STL · GitHub
Skip to content

<optional>: Remove stale LWG-3627 comments - #6416

Open
zika (tycket033-tech) wants to merge 1 commit into
microsoft:mainfrom
tycket033-tech:codex/issue-5851-remove-lwg-3627-comments
Open

<optional>: Remove stale LWG-3627 comments#6416
zika (tycket033-tech) wants to merge 1 commit into
microsoft:mainfrom
tycket033-tech:codex/issue-5851-remove-lwg-3627-comments

Conversation

@tycket033-tech

Copy link
Copy Markdown
Contributor

The resolution for LWG-3627 was already implemented in #2607, so the remaining
comments are no longer needed.

This PR removes the two stale LWG-3627 comments from <optional> and its
test. There are no behavior changes.

No tests were run because this is a comment-only change.

Fixes #5851.

AI assistance: I used Codex to find the comments and help prepare this PR.
I've reviewed and understand the change.

@azure-pipelines

This comment was marked as resolved.

@tycket033-tech
zika (tycket033-tech) marked this pull request as ready for review August 24, 2026 14:54
@tycket033-tech
zika (tycket033-tech) requested a review from a team as a code owner August 24, 2026 14:54
Copilot AI balanced review requested due to automatic review settings August 24, 2026 14:54
@azure-pipelines

This comment was marked as resolved.

This comment was marked as resolved.

@tycket033-tech

Copy link
Copy Markdown
Contributor Author

@frederick-vs-ja

Copy link
Copy Markdown
Contributor

@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) added documentation Related to documentation or comments LWG Library Working Group issue labels Aug 31, 2026
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) moved this from Initial Review to Ready To Merge in STL Code Reviews Sep 3, 2026
@StephanTLavavej

This comment was marked as resolved.

@azure-pipelines

This comment was marked as resolved.

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

Labels

documentation Related to documentation or comments LWG Library Working Group issue

Projects

Status: Ready To Merge

Development

Successfully merging this pull request may close these issues.

LWG-3627 Inconsistent specifications for std::make_optional overloads

4 participants