Skip to content

Conversation

@justinegeffen
Copy link
Contributor

@justinegeffen justinegeffen commented May 1, 2025

Removed an internal note from legacy content. We may still want to provide the images referenced in the note but we can create a Jira issue for that.

No formatting was changed in these pages, but we should consider reformatting them.

Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
@justinegeffen justinegeffen added the 1. Editor review Needs a language review label May 1, 2025
@netlify
Copy link

netlify bot commented May 1, 2025

Deploy Preview for seqera-docs ready!

Name Link
🔨 Latest commit 4b64340
🔍 Latest deploy log https://app.netlify.com/projects/seqera-docs/deploys/6874b94dba7dd40008c90ab2
😎 Deploy Preview https://deploy-preview-578--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.

@justinegeffen justinegeffen requested a review from llewellyn-sl May 1, 2025 12:59
Copy link
Member

@christopher-hakkaart christopher-hakkaart 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 there are other changes required.

Line 14 looks like AWS should be a heading, with text below. Similarly, for line 55 (Azure) and 83 (Google cloud).

Also note that the numbering for Google cloud starts at 5.

@justinegeffen
Copy link
Contributor Author

@llewellyn-sl, these pages don't seem to be in the LHS in prod but they're still part of the doc set (e.g., https://deploy-preview-578--seqera-docs.netlify.app/platform-enterprise/24.1/enterprise/advanced-topics/seqera-container-images). Are we still using this content?

…ges.md

Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
…ges.md

Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
…ges.md

Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
…ges.md

Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
…ges.md

Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1. Editor review Needs a language review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants