Skip to content

Conversation

@insukcho
Copy link
Contributor

@insukcho insukcho commented Nov 28, 2025

Fixing two invalid links from azure to gcp

Summary

Generative AI disclosure

  1. Did you use a generative AI (GenAI) tool to assist in creating this contribution?
  • Yes
  • No

Fixing two invalid links from `azure` to `gcp`
@insukcho insukcho requested a review from a team as a code owner November 28, 2025 17:23
@github-actions
Copy link

github-actions bot commented Nov 28, 2025

✅ Vale Linting Results

No issues found on modified lines!

@github-actions
Copy link

github-actions bot commented Nov 28, 2025

🔍 Preview links for changed docs

Copy link
Contributor

@florent-leborgne florent-leborgne left a comment

Choose a reason for hiding this comment

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

I think these are supposed to keep users on the same page since these sections are visibly there too. The current links and the suggested fixes send users to a different page, which looks strange to me, and if correct would make me question why do we have those sections after on the page :D
Thanks for finding them and attempting a fix.

Copy link
Contributor

@benironside benironside left a comment

Choose a reason for hiding this comment

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

Tested the links in the preview and they're working

@benironside benironside dismissed florent-leborgne’s stale review December 1, 2025 22:55

Dismissing florent's review because I incorporated his suggestion and the PR now works as expected.

@benironside benironside enabled auto-merge (squash) December 1, 2025 22:55
@benironside benironside merged commit e66bd48 into main Dec 1, 2025
8 checks passed
@benironside benironside deleted the insukcho-patch-3 branch December 1, 2025 22:57
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.

4 participants