Skip to content

Conversation

Rageking8
Copy link
Contributor

Prefer https links where possible.

Copy link
Contributor

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

Copy link
Contributor

Learn Build status updates of commit 5f82740:

✅ Validation status: passed

File Status Preview URL Details
docs/code-quality/using-the-cpp-core-guidelines-checkers.md ✅Succeeded
docs/error-messages/compiler-errors-2/compiler-error-c3066.md ✅Succeeded
docs/overview/cpp-conformance-improvements-2017.md ✅Succeeded
docs/overview/what-s-new-for-visual-cpp-in-visual-studio.md ✅Succeeded

For more details, please refer to the build report.

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates HTTP links to HTTPS where possible to improve security and follow best practices for web links in documentation.

Key changes:

  • Updated three HTTP links to HTTPS equivalents
  • Modified links in comments and documentation text

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
docs/overview/what-s-new-for-visual-cpp-in-visual-studio.md Updated WG21 link from HTTP to HTTPS
docs/overview/cpp-conformance-improvements-2017.md Updated open-std.org link from HTTP to HTTPS
docs/error-messages/compiler-errors-2/compiler-error-c3066.md Updated open-std.org link from HTTP to HTTPS
docs/code-quality/using-the-cpp-core-guidelines-checkers.md Updated NuGet package link from HTTP to HTTPS

@v-dirichards
Copy link
Contributor

@TylerMSFT

Can you review the proposed changes?

Editorial changes LGTM

Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged Tracking label for the PR review team label Sep 29, 2025
Copy link
Collaborator

@TylerMSFT TylerMSFT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I feel more secure now :-)

@TylerMSFT
Copy link
Collaborator

#sign-off

@v-dirichards v-dirichards merged commit 9a0eec0 into MicrosoftDocs:main Sep 30, 2025
2 checks passed
@Rageking8 Rageking8 deleted the update-some-http-links-to-https branch September 30, 2025 23:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants