Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

docs: restructure getting-started section #2294

Merged
merged 8 commits into from
Oct 24, 2023
Merged

docs: restructure getting-started section #2294

merged 8 commits into from
Oct 24, 2023

Conversation

StackScribe
Copy link
Contributor

@StackScribe StackScribe commented Oct 18, 2023 •
edited by mowies
Loading

Fixes #2203

This PR:

  • Moves the getting started guides for Metrics, Observability, and Release Lifecycle Management into the "Getting Started" section of the docs
  • Fixed the xrefs that were broken by moving the files
  • Minimal modifications were made to the text of these pages.
  • The individual pages will be enhanced in separate PRs

@StackScribe StackScribe added the documentation Improvements or additions to documentation label Oct 18, 2023
@StackScribe StackScribe added this to the 0.9 milestone Oct 18, 2023
@StackScribe StackScribe self-assigned this Oct 18, 2023
@StackScribe StackScribe requested review from a team as code owners October 18, 2023 11:59
@netlify
Copy link

netlify bot commented Oct 18, 2023 •
edited
Loading

✅ Deploy Preview for keptn-lifecycle-toolkit ready!

Name Link
🔨 Latest commit 0206822
🔍 Latest deploy log https://app.netlify.com/sites/keptn-lifecycle-toolkit/deploys/65378550b5980a0008186d56
😎 Deploy Preview https://deploy-preview-2294--keptn-lifecycle-toolkit.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 site configuration.

@StackScribe StackScribe mentioned this pull request Oct 18, 2023
Closed
@StackScribe
fix xrefs
4aa6bd7 
Signed-off-by: Meg McRoberts <[email protected]>
@StackScribe
more xrefs
c37b25a 
Signed-off-by: Meg McRoberts <[email protected]>
@StackScribe
yet another xref
b1d1979 
Signed-off-by: Meg McRoberts <[email protected]>
@StackScribe
Copy link
Contributor Author

@prakrit55

@StackScribe
Copy link
Contributor Author

This is supposed to be a structure-only PR that got merged very quickly so that I could go in and work on issues like the ones @odubajDT has flagged here. I am blocked until this PR is merged so let's discuss the best way to proceed here.

@mowies
Copy link
Member

mowies commented Oct 23, 2023

#2216 is only talking about the landing page, why is this related to it?

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@mowies
Copy link
Member

mowies commented Oct 24, 2023

Fixes #2203

@odubajDT odubajDT merged commit 8c56087 into keptn:main Oct 24, 2023
18 checks passed
@keptn-bot keptn-bot mentioned this pull request Oct 24, 2023
Merged
@keptn-bot keptn-bot mentioned this pull request Jan 23, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@odubajDT odubajDT odubajDT approved these changes

@mowies mowies mowies approved these changes

Assignees

@StackScribe StackScribe

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
0.9
Development

Successfully merging this pull request may close these issues.

Move all "Getting started" guides into the "Getting started" section and delete "Tutorials" sections
3 participants
@StackScribe @mowies @odubajDT