Skip to content

Maintenance: extract docs/snippets linting in its own CI step #1275

Closed
@dreamorosi

Description

@dreamorosi

Summary

#1259 introduced changes that allow us to automatically lint the contents of the code snippets present in the docs. When redacting the PR, these checks were conflated as part of the unit tests/linting for the core utilities. This is fine, however if the linting of these code snippets fails, the failure would be opaque for the reviewers as it would appear as part of the main checks (see img below):
image

Why is this needed?

For a better and more granular at-a-glance view for both maintainers and PR authors, it would be great to have these checks in a separate item/steps so that it's immediately clear where the failure happened.

Which area does this relate to?

Automation

Solution

Update the workflow that runs the tests & linting so that the check is extracted and shows up as a separate item in the PR view:
image

Acknowledgment

Metadata

Metadata

Assignees

Labels

automationThis item relates to automationcompletedThis item is complete and has been merged/shippedinternalPRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.)

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions