Skip to content

Tests to validate CSS used by components#1608

Merged
mxriverlynn merged 15 commits intomainfrom
mxriverlynn/validate-component-css-use
Nov 17, 2022
Merged

Tests to validate CSS used by components#1608
mxriverlynn merged 15 commits intomainfrom
mxriverlynn/validate-component-css-use

Conversation

@mxriverlynn
Copy link
Contributor

@mxriverlynn mxriverlynn commented Nov 16, 2022

Description

this PR adds CSS tests from the opposite direction of #1574, by looking for CSS classes that are used in PVC components and ensuring they are valid. to do that, it checks against the same css source as the previous PR, to find the classes in question.

Integration

Does this change require any updates to code in production?

no

Merge checklist

  • Added/updated tests
  • Added/updated documentation
  • Added/updated previews

@changeset-bot
Copy link

changeset-bot bot commented Nov 16, 2022

🦋 Changeset detected

Latest commit: 3087ae6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@primer/view-components Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@mxriverlynn mxriverlynn temporarily deployed to review-pr-1608 November 16, 2022 14:39 Inactive
@mxriverlynn mxriverlynn temporarily deployed to github-pages November 16, 2022 14:44 Inactive
@mxriverlynn mxriverlynn temporarily deployed to review-pr-1608 November 16, 2022 19:49 Inactive
@mxriverlynn mxriverlynn temporarily deployed to github-pages November 16, 2022 19:55 Inactive
@mxriverlynn mxriverlynn temporarily deployed to review-pr-1608 November 17, 2022 18:17 Inactive
@mxriverlynn mxriverlynn temporarily deployed to github-pages November 17, 2022 18:21 Inactive
… moved into PVC. also fixed when the assertion is made, so it catches all errors
@github-actions
Copy link
Contributor

github-actions bot commented Nov 17, 2022

⚠️ Visual differences found

Our visual comparison tests found UI differences. Please review the differences by viewing the files changed tab to ensure that the changes were intentional.

Review visual differences

@mxriverlynn mxriverlynn force-pushed the mxriverlynn/validate-component-css-use branch from 30d61ee to ebe6567 Compare November 17, 2022 19:17
@mxriverlynn mxriverlynn temporarily deployed to review-pr-1608 November 17, 2022 19:17 Inactive
@mxriverlynn mxriverlynn marked this pull request as ready for review November 17, 2022 19:20
@mxriverlynn mxriverlynn requested review from a team and jonrohan November 17, 2022 19:20
@mxriverlynn mxriverlynn force-pushed the mxriverlynn/validate-component-css-use branch from 8030e99 to 3087ae6 Compare November 17, 2022 19:33
@mxriverlynn mxriverlynn requested a review from jonrohan November 17, 2022 19:35
@mxriverlynn mxriverlynn temporarily deployed to github-pages November 17, 2022 19:38 Inactive
@mxriverlynn mxriverlynn merged commit 75ad476 into main Nov 17, 2022
@mxriverlynn mxriverlynn deleted the mxriverlynn/validate-component-css-use branch November 17, 2022 20:55
@primer-css primer-css mentioned this pull request Nov 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants