Skip to content

Conversation

@PixelRobots
Copy link
Contributor

@PixelRobots PixelRobots commented Jul 29, 2025

Purpose

Add link back to setting up the lab environment.

Does this introduce a breaking change?

[ ] Yes
[ x] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

Copilot AI review requested due to automatic review settings July 29, 2025 07:13
@PixelRobots PixelRobots requested a review from chzbrgr71 as a code owner July 29, 2025 07:13
Copy link
Contributor

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 restores a missing link to the "Setting Up Lab Environment" guide in the Istio lab documentation. The change improves navigation by providing users with a direct link to the prerequisite setup documentation instead of just referencing it without a link.

  • Adds a clickable link to the lab environment setup guide in the prerequisites section

@pauldotyu
Copy link
Collaborator

Thanks for reviewing the lab @PixelRobots!

I think I see what happened here. @dcasati updated the lab to include cluster setup instructions in the Step 2: Create a new resource group section so the text instructing a learner to "Follow the guide here:" should be removed altogether.

Eventually, this lab should be updated to use the new shared markdown components to include consistent cluster setup instructions (inline) without having the learner leave the page and come back.

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.

2 participants