Skip to content

Revert "Temporarily disable warning C5387 on MSVC" #14103

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
Draft
wants to merge 1 commit into
base: trunk
Choose a base branch
from
Draft

Revert "Temporarily disable warning C5387 on MSVC" #14103

wants to merge 1 commit into from

Conversation

dra27
Copy link
Member

@dra27 dra27 commented Jun 23, 2025

TODO item following up #14102 to be merged as-and-when:

  1. https://developercommunity.visualstudio.com/t/warning-C5287:-operands-are-different-e/10877942#T-N10914938 is released and renders the disabling of C5387 unnecessary
  2. GitHub Actions runners are updated to use that release

@dra27 dra27 added no-change-entry-needed CI CI: Full matrix Makes the CI test a bigger set of configurations labels Jun 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@dra27 dra27

Labels
CI: Full matrix Makes the CI test a bigger set of configurations CI no-change-entry-needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dra27