Skip to content

chore: Drop Use IAM roles guide, replaced by inline recommendations#1102

Merged
justinegeffen merged 2 commits intomasterfrom
chiusole/remove-unused-iam-role-pages
Feb 5, 2026
Merged

chore: Drop Use IAM roles guide, replaced by inline recommendations#1102
justinegeffen merged 2 commits intomasterfrom
chiusole/remove-unused-iam-role-pages

Conversation

@bebosudo
Copy link
Member

@bebosudo bebosudo commented Feb 5, 2026

In the process of removing mentions of the nf-tower-aws repo, I found this docs page not linked to from any v25.x page anymore, and already removed from cloud docs: the content in the page is already being provided in a section of the AWS Batch guide.

@bebosudo bebosudo changed the title Drop Use IAM roles guide, replaced by inline recommendations chore: Drop Use IAM roles guide, replaced by inline recommendations Feb 5, 2026
@netlify
Copy link

netlify bot commented Feb 5, 2026

Deploy Preview for seqera-docs ready!

Name Link
🔨 Latest commit 4aa3d5c
🔍 Latest deploy log https://app.netlify.com/projects/seqera-docs/deploys/6984accaffba070007539bc6
😎 Deploy Preview https://deploy-preview-1102--seqera-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

Editorial Review Complete - Reviewed changed files. The editorial agents found some potential improvements, but they're all on lines outside the PR diff. The changed lines look good! Review by Claude Code editorial agents

@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

📝 Documentation Review Summary (PR type: content)

Check Status
Voice/Tone ✅ success
Terminology ✅ success

Review powered by Claude Code editorial agents

To apply suggestions:

  • Click "Commit suggestion" on individual inline comments
  • Select multiple suggestions and batch commit them together

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 removes the "Use AWS IAM roles" guide from documentation versions 25.1, 25.2, 25.3, and current, as the content has been replaced by inline recommendations in the AWS Batch compute environment guide. The guide references external resources from the deprecated nf-tower-aws repository and is no longer linked from v25.x documentation pages.

Changes:

  • Removed use-iam-role.md files from versions 25.1, 25.2, 25.3, and current documentation
  • Removed sidebar references to the use-iam-role guide from the same versions

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
platform-enterprise_versioned_sidebars/version-25.3-sidebars.json Removes use-iam-role entry from Advanced topics sidebar
platform-enterprise_versioned_sidebars/version-25.2-sidebars.json Removes use-iam-role entry from Advanced topics sidebar
platform-enterprise_versioned_sidebars/version-25.1-sidebars.json Removes use-iam-role entry from Advanced topics sidebar
platform-enterprise_versioned_docs/version-25.3/enterprise/advanced-topics/use-iam-role.md Deletes the IAM roles guide file
platform-enterprise_versioned_docs/version-25.2/enterprise/advanced-topics/use-iam-role.md Deletes the IAM roles guide file
platform-enterprise_versioned_docs/version-25.1/enterprise/advanced-topics/use-iam-role.md Deletes the IAM roles guide file
platform-enterprise_docs/enterprise/advanced-topics/use-iam-role.md Deletes the IAM roles guide file from current docs
platform-enterprise_docs/enterprise-sidebar.json Removes use-iam-role entry from current docs sidebar

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@justinegeffen justinegeffen added 1. Editor review Needs a language review 1. Dev/PM/SME Needs a review by a Dev/PM/SME content-improvement This work improves content editorially or structurally. labels Feb 5, 2026
@justinegeffen justinegeffen merged commit 340bba7 into master Feb 5, 2026
10 checks passed
@justinegeffen justinegeffen deleted the chiusole/remove-unused-iam-role-pages branch February 5, 2026 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1. Dev/PM/SME Needs a review by a Dev/PM/SME 1. Editor review Needs a language review content-improvement This work improves content editorially or structurally.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants