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-6238 Add Test Visibility and Intelligent Test Runner Documentation #20944

Merged
merged 6 commits into from
Dec 14, 2023
Merged

DOCS-6238 Add Test Visibility and Intelligent Test Runner Documentation #20944

merged 6 commits into from
Dec 14, 2023

Conversation

alai97
Copy link
Contributor

@alai97 alai97 commented Dec 7, 2023 •
edited
Loading

What does this PR do? What is the motivation?

Extracts Test Visibility and ITR documentation from CI Visibility doc set along with guides and troubleshooting content. Cleans up legacy links.

Merge instructions

  • Please merge after reviewing

Additional notes

@alai97 alai97 requested review from a team as code owners December 7, 2023 22:30
@github-actions github-actions bot added the Architecture Everything related to the Doc backend label Dec 7, 2023
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Dec 7, 2023 •
edited
Loading

Preview links (active after the build_preview check completes)

New or renamed files

Renamed files

Modified Files

@github-actions github-actions bot added the Guide Content impacting a guide label Dec 7, 2023
@alai97 alai97 changed the title Add Test Visibility and Intelligent Test Runner Documentation DOCS-6238 Add Test Visibility and Intelligent Test Runner Documentation Dec 7, 2023
@jhgilbert jhgilbert added the editorial review Waiting on a more in-depth review label Dec 7, 2023
drichards-87
drichards-87 requested changes Dec 11, 2023
View reviewed changes
Copy link
Contributor

@drichards-87 drichards-87 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 overall! I just had a few minor suggestions + I noted a couple of broken links.

content/en/tests/developer_workflows.md Outdated Show resolved Hide resolved
content/en/tests/containers.md Outdated Show resolved Hide resolved
content/en/tests/code_coverage.md Outdated Show resolved Hide resolved
content/en/tests/browser_tests.md Outdated Show resolved Hide resolved
content/en/tests/browser_tests.md Outdated Show resolved Hide resolved
content/en/tests/_index.md Outdated Show resolved Hide resolved
content/en/tests/_index.md Outdated Show resolved Hide resolved
content/en/intelligent_test_runner/troubleshooting/_index.md Outdated Show resolved Hide resolved
@alai97 @drichards-87
Doc Review
9d0e64d 
Co-authored-by: DeForest Richards <56796055+drichards-87@users.noreply.github.com>
drichards-87
drichards-87 approved these changes Dec 12, 2023
View reviewed changes
Copy link
Contributor

@drichards-87 drichards-87 left a comment

Choose a reason for hiding this comment

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

Noticed one last issue with a link (it was missing /setup/ in the URL path). Other than that, it looks good! PR is approved.

content/en/tests/_index.md Outdated Show resolved Hide resolved
romainkomorndatadog
romainkomorndatadog approved these changes Dec 12, 2023
View reviewed changes
Copy link
Contributor

@romainkomorndatadog romainkomorndatadog left a comment

Choose a reason for hiding this comment

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

This looks good to me from the libraries side, but I'd like to defer to @KickingTheTV for the way this makes ITR a product separate from test visibility, so my approval's contingent on his approval.

@alai97 @drichards-87
One More
c882778 
Co-authored-by: DeForest Richards <56796055+drichards-87@users.noreply.github.com>
KickingTheTV
KickingTheTV approved these changes Dec 14, 2023
View reviewed changes
Copy link
Contributor

@KickingTheTV KickingTheTV left a comment

Choose a reason for hiding this comment

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

LGTM

@alai97 alai97 merged commit dfde9c2 into alai97/software-delivery-renaming Dec 14, 2023
8 of 10 checks passed
@alai97 alai97 deleted the alai97/move-tests-and-itr branch December 14, 2023 18:18
alai97 added a commit that referenced this pull request Dec 14, 2023
@alai97 @drichards-87
DOCS-6873 Rename "Continuous Integration" to "Software Delivery" (#20816
cc4edf0 
)

* Rename CI to Software Delivery

* Reorder ITR

* Spacing Nit

* Update Slugs + Add Aliases

* Update .gitignore

* Revert "Update Slugs + Add Aliases"

This reverts commit 6739086.

* Cleanup Slugs + Add Aliases

* Generate Preview

* Update Static Analysis Rules

* DOCS-6238 Add Test Visibility and Intelligent Test Runner Documentation (#20944)

* Make Individual Doc Sets

* Separate Guide + Troubleshooting Content

* More Link Fixes

* Naming Nit

* Doc Review

Co-authored-by: DeForest Richards <56796055+drichards-87@users.noreply.github.com>

* One More

Co-authored-by: DeForest Richards <56796055+drichards-87@users.noreply.github.com>

---------

Co-authored-by: DeForest Richards <56796055+drichards-87@users.noreply.github.com>

* DOCS-6873 Static Analysis URL Update (#20913)

* Update CODEOWNERS + Doc Links

* Link Fix

* Ready to Merge

---------

Co-authored-by: DeForest Richards <56796055+drichards-87@users.noreply.github.com>
alai97 added a commit that referenced this pull request Dec 14, 2023
@alai97 @andrea-mosk
@estherk15 @drichards-87
DOCS-5876 Adds CD Visibility + ArgoCD Documentation (#20798)
76e0c24 
* DOCS-5876

Updates the navigation for the Docs Site + a doc set for CD Visibility.

* Add Beta Banner

* Fill In ArgoCD Information

* Link Fix

* Adding initial setup of Argo CD

* Add Explorer + Search Deployments Info and Screenshots

* Wording Nit

* Addressing PR Feedback

* Replacing create with add

* Small fixes

* Adding application subscription guide

* Remove Placeholder Stuff

* Moving sentence below

* Doc Review

Co-authored-by: andrea-mosk <39099407+andrea-mosk@users.noreply.github.com>
Co-authored-by: Esther Kim <esther.kim@datadoghq.com>

* Clarify duration facet example

* Clarify duration facet example for CI

* SME Review

Co-authored-by: andrea-mosk <39099407+andrea-mosk@users.noreply.github.com>
Co-authored-by: Esther Kim <esther.kim@datadoghq.com>

* Updating images

* Updating facets image

* Link Nit

Co-authored-by: andrea-mosk <39099407+andrea-mosk@users.noreply.github.com>

* DOCS-6873 Rename "Continuous Integration" to "Software Delivery" (#20816)

* Rename CI to Software Delivery

* Reorder ITR

* Spacing Nit

* Update Slugs + Add Aliases

* Update .gitignore

* Revert "Update Slugs + Add Aliases"

This reverts commit 6739086.

* Cleanup Slugs + Add Aliases

* Generate Preview

* Update Static Analysis Rules

* DOCS-6238 Add Test Visibility and Intelligent Test Runner Documentation (#20944)

* Make Individual Doc Sets

* Separate Guide + Troubleshooting Content

* More Link Fixes

* Naming Nit

* Doc Review

Co-authored-by: DeForest Richards <56796055+drichards-87@users.noreply.github.com>

* One More

Co-authored-by: DeForest Richards <56796055+drichards-87@users.noreply.github.com>

---------

Co-authored-by: DeForest Richards <56796055+drichards-87@users.noreply.github.com>

* DOCS-6873 Static Analysis URL Update (#20913)

* Update CODEOWNERS + Doc Links

* Link Fix

* Ready to Merge

---------

Co-authored-by: DeForest Richards <56796055+drichards-87@users.noreply.github.com>

---------

Co-authored-by: Andrea Moscatelli <andrea.moscatelli@datadoghq.com>
Co-authored-by: andrea-mosk <39099407+andrea-mosk@users.noreply.github.com>
Co-authored-by: Esther Kim <esther.kim@datadoghq.com>
Co-authored-by: DeForest Richards <56796055+drichards-87@users.noreply.github.com>
MaelNamNam pushed a commit that referenced this pull request Jan 17, 2024
@alai97 @andrea-mosk
@estherk15 @drichards-87 @MaelNamNam
DOCS-5876 Adds CD Visibility + ArgoCD Documentation (#20798)
619fa01 
* DOCS-5876

Updates the navigation for the Docs Site + a doc set for CD Visibility.

* Add Beta Banner

* Fill In ArgoCD Information

* Link Fix

* Adding initial setup of Argo CD

* Add Explorer + Search Deployments Info and Screenshots

* Wording Nit

* Addressing PR Feedback

* Replacing create with add

* Small fixes

* Adding application subscription guide

* Remove Placeholder Stuff

* Moving sentence below

* Doc Review

Co-authored-by: andrea-mosk <39099407+andrea-mosk@users.noreply.github.com>
Co-authored-by: Esther Kim <esther.kim@datadoghq.com>

* Clarify duration facet example

* Clarify duration facet example for CI

* SME Review

Co-authored-by: andrea-mosk <39099407+andrea-mosk@users.noreply.github.com>
Co-authored-by: Esther Kim <esther.kim@datadoghq.com>

* Updating images

* Updating facets image

* Link Nit

Co-authored-by: andrea-mosk <39099407+andrea-mosk@users.noreply.github.com>

* DOCS-6873 Rename "Continuous Integration" to "Software Delivery" (#20816)

* Rename CI to Software Delivery

* Reorder ITR

* Spacing Nit

* Update Slugs + Add Aliases

* Update .gitignore

* Revert "Update Slugs + Add Aliases"

This reverts commit 6739086.

* Cleanup Slugs + Add Aliases

* Generate Preview

* Update Static Analysis Rules

* DOCS-6238 Add Test Visibility and Intelligent Test Runner Documentation (#20944)

* Make Individual Doc Sets

* Separate Guide + Troubleshooting Content

* More Link Fixes

* Naming Nit

* Doc Review

Co-authored-by: DeForest Richards <56796055+drichards-87@users.noreply.github.com>

* One More

Co-authored-by: DeForest Richards <56796055+drichards-87@users.noreply.github.com>

---------

Co-authored-by: DeForest Richards <56796055+drichards-87@users.noreply.github.com>

* DOCS-6873 Static Analysis URL Update (#20913)

* Update CODEOWNERS + Doc Links

* Link Fix

* Ready to Merge

---------

Co-authored-by: DeForest Richards <56796055+drichards-87@users.noreply.github.com>

---------

Co-authored-by: Andrea Moscatelli <andrea.moscatelli@datadoghq.com>
Co-authored-by: andrea-mosk <39099407+andrea-mosk@users.noreply.github.com>
Co-authored-by: Esther Kim <esther.kim@datadoghq.com>
Co-authored-by: DeForest Richards <56796055+drichards-87@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@KickingTheTV KickingTheTV KickingTheTV approved these changes

@drichards-87 drichards-87 drichards-87 approved these changes

@romainkomorndatadog romainkomorndatadog romainkomorndatadog approved these changes

Assignees
No one assigned
Labels
Architecture Everything related to the Doc backend editorial review Waiting on a more in-depth review Guide Content impacting a guide
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@alai97 @KickingTheTV @drichards-87 @romainkomorndatadog @jhgilbert