Skip to content

Conversation

@pose
Copy link
Contributor

@pose pose commented Oct 22, 2025

Fix broken link

Fix broken link
@pose pose requested review from a team and pulumi-bot October 22, 2025 11:26
@claude
Copy link
Contributor

claude bot commented Oct 22, 2025

Review Summary

Link formatting fix is correct - The PR properly fixes the broken Markdown link formatting by moving the URL to the same line as the link text (line 59).

Optional Improvement

While the link will work due to Hugo aliases, consider updating to the canonical URL path:

Line 59:

The full import file schema references can be found in the [import documentation](https://www.pulumi.com/docs/iac/guides/migration/import/#bulk-import-operations).

The path /docs/iac/adopting-pulumi/import/ is an old URL that redirects via alias to /docs/iac/guides/migration/import/. Using the canonical path directly is preferred for clarity and avoids unnecessary redirects.

What I Checked

  • ✅ Markdown link syntax is now correct
  • ✅ Link target exists and resolves (via alias)
  • ✅ Anchor #bulk-import-operations correctly references the "Bulk Import Operations" heading
  • ✅ No spelling or grammar issues
  • ✅ File ends with newline

Note: If you'd like me to make the canonical URL update or perform additional reviews, feel free to mention me (@claude) in a comment.

@pulumi-bot
Copy link
Collaborator

@CamSoper
Copy link
Contributor

@claude Please go ahead and update to use the canonical URL.

@claude
Copy link
Contributor

claude bot commented Oct 22, 2025

Claude finished @CamSoper's task —— View job


Review Summary

Link formatting fix is correct - The PR properly fixes the broken Markdown link formatting by moving the URL to the same line as the link text (line 59).

Update Complete

  • Read current file content
  • Update link to use canonical URL path
  • Commit and push changes

I've updated the link on line 59 to use the canonical URL path /docs/iac/guides/migration/import/ instead of the old /docs/iac/adopting-pulumi/import/ path. The change has been committed and pushed to the pose-patch-1 branch.

Changes made:

  • Updated content/docs/iac/cli/commands/pulumi_import.md:59 to use the canonical URL path

Job RunBranch: pose-patch-1

Use /docs/iac/guides/migration/import/ instead of the old
/docs/iac/adopting-pulumi/import/ path which redirects via alias.

Co-authored-by: Cam Soper <CamSoper@users.noreply.github.com>
@pulumi-bot
Copy link
Collaborator

Copy link
Contributor

@CamSoper CamSoper left a comment

Choose a reason for hiding this comment

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

Thanks for doing this! Looks good!

@CamSoper CamSoper enabled auto-merge (squash) October 22, 2025 16:41
@pulumi-bot
Copy link
Collaborator

@CamSoper CamSoper merged commit c75b0cc into master Oct 22, 2025
8 checks passed
@CamSoper CamSoper deleted the pose-patch-1 branch October 22, 2025 16:49
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.

3 participants