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-6279 Adds Getting Started with CI Visibility Documentation #22692

Merged
merged 10 commits into from
May 2, 2024
Merged

DOCS-6279 Adds Getting Started with CI Visibility Documentation #22692

merged 10 commits into from
May 2, 2024

Conversation

alai97
Copy link
Contributor

@alai97 alai97 commented Apr 16, 2024

What does this PR do? What is the motivation?

Adds a Getting Started with CI Visibility doc page + screenshots to the Getting Started section of the Docs Site.

Merge instructions

  • Please merge after reviewing

Additional notes

@alai97
DOCS-6279
2b788e7 
Adds Getting Started with CI Visibility doc page + screenshots.
@alai97 alai97 requested a review from kassenq April 16, 2024 23:23
@alai97 alai97 requested review from a team as code owners April 16, 2024 23:23
@github-actions github-actions bot added Architecture Everything related to the Doc backend Images Images are added/removed with this PR labels Apr 16, 2024
@github-actions GitHub Actions
Copy link
Contributor

Preview links (active after the build_preview check completes)

New or renamed files

Modified Files

@alai97 alai97 added the editorial review Waiting on a more in-depth review label Apr 16, 2024
@alai97 alai97 requested a review from KickingTheTV April 17, 2024 16:20
rtrieu
rtrieu requested changes Apr 26, 2024
View reviewed changes
Copy link
Contributor

@rtrieu rtrieu left a comment

Choose a reason for hiding this comment

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

Left some feedback for your review, but otherwise looks good!

content/en/getting_started/ci_visibility/_index.md Outdated Show resolved Hide resolved
content/en/getting_started/ci_visibility/_index.md Outdated Show resolved Hide resolved
content/en/getting_started/ci_visibility/_index.md Outdated Show resolved Hide resolved
content/en/getting_started/ci_visibility/_index.md Outdated Show resolved Hide resolved
@alai97 @rtrieu
Doc Review
3c7d5d3 
Co-authored-by: Rosa Trieu <[email protected]>
@alai97 alai97 added the WORK IN PROGRESS No review needed, it's a wip ;) label Apr 26, 2024
@alai97 alai97 requested a review from rtrieu April 26, 2024 21:41
@alai97 alai97 removed the WORK IN PROGRESS No review needed, it's a wip ;) label Apr 26, 2024
rtrieu
rtrieu approved these changes Apr 29, 2024
View reviewed changes
Copy link
Contributor

@rtrieu rtrieu left a comment

Choose a reason for hiding this comment

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

Looks good!

@alai97 alai97 merged commit af5754f into master May 2, 2024
14 checks passed
@alai97 alai97 deleted the alai97/add-getting-started-with-pipeline-visibility branch May 2, 2024 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rtrieu rtrieu rtrieu approved these changes

@KickingTheTV KickingTheTV KickingTheTV approved these changes

@kassenq kassenq Awaiting requested review from kassenq

Assignees
No one assigned
Labels
Architecture Everything related to the Doc backend editorial review Waiting on a more in-depth review Images Images are added/removed with this PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@alai97 @KickingTheTV @rtrieu