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.
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
DOCS-7722 CI Visibility and Test Visibility Product Landing Page Edits #23064
base: master
Are you sure you want to change the base?
Conversation
Preview links (active after the
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, approving with a suggestion to break up a long sentence.
Co-authored-by: Bryce Eadie <bryce.eadie@datadoghq.com>
Co-authored-by: Nikita Tkachenko <121111529+nikita-tkachenko-datadog@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything LGTM
But can we just make one change while we're here. For the test setup page where we have the table of features and compatibility, can we add a checkmark for Intelligent Test Runner for Ruby? We now support ITR for all languages
What does this PR do? What is the motivation?
Now that we've separated CI Visibility into CI Pipeline Visibility and Test Visibility, it's time to distinguish the pipeline content from the test content on the product landing pages. We're scrubbing any mention of "tests" from CI Visibility, and making sure that information is covered on the Test Visibility landing page (along with an update to the language/options partial).
Merge instructions
Additional notes