Skip to content

Conversation

agreenberry
Copy link
Contributor

@agreenberry agreenberry commented Jan 11, 2023

Describe your changes here.

Closes #1572

Screenshots

When implementing a Heading component without the relevant heading element type, like so...
image

...Then, the user sees as error in the code, as well as a log to warn the developer again:
image

Merge checklist

  • Added/updated tests
  • Added/updated documentation
  • Tested in Chrome
  • Tested in Firefox
  • Tested in Safari
  • Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

@changeset-bot
Copy link

changeset-bot bot commented Jan 11, 2023

🦋 Changeset detected

Latest commit: a844495

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

This PR includes changesets to release 1 package
Name Type
@primer/react Minor

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

@github-actions
Copy link
Contributor

github-actions bot commented Jan 11, 2023

size-limit report 📦

Path Size
dist/browser.esm.js 89.03 KB (+0.1% 🔺)
dist/browser.umd.js 89.64 KB (+0.09% 🔺)

@agreenberry agreenberry temporarily deployed to github-pages January 11, 2023 23:11 — with GitHub Actions Inactive
@github-actions github-actions bot temporarily deployed to storybook-preview-2768 January 11, 2023 23:11 Inactive
@agreenberry agreenberry force-pushed the 1572-accessibility-request-primer-react-heading-enforce-what-as-can-be-set-to-with-a-runtime-validation-rule branch from 860946a to 37ba4ef Compare January 24, 2023 16:30
…ing-enforce-what-as-can-be-set-to-with-a-runtime-validation-rule
@github-actions github-actions bot temporarily deployed to storybook-preview-2768 January 24, 2023 16:37 Inactive
@agreenberry agreenberry marked this pull request as ready for review January 24, 2023 16:41
@agreenberry agreenberry requested review from a team and josepmartins January 24, 2023 16:41
@github-actions github-actions bot temporarily deployed to storybook-preview-2768 January 24, 2023 16:41 Inactive
@github-actions github-actions bot temporarily deployed to storybook-preview-2768 January 24, 2023 16:46 Inactive
@agreenberry agreenberry temporarily deployed to github-pages January 24, 2023 16:50 — with GitHub Actions Inactive
@github-actions github-actions bot temporarily deployed to storybook-preview-2768 January 24, 2023 16:50 Inactive
@agreenberry agreenberry temporarily deployed to github-pages January 24, 2023 17:53 — with GitHub Actions Inactive
@github-actions github-actions bot temporarily deployed to storybook-preview-2768 January 24, 2023 17:53 Inactive
…ing-enforce-what-as-can-be-set-to-with-a-runtime-validation-rule
@agreenberry agreenberry temporarily deployed to github-pages January 27, 2023 16:33 — with GitHub Actions Inactive
@github-actions github-actions bot temporarily deployed to storybook-preview-2768 January 27, 2023 16:33 Inactive
…ing-enforce-what-as-can-be-set-to-with-a-runtime-validation-rule
@agreenberry agreenberry temporarily deployed to github-pages January 27, 2023 18:45 — with GitHub Actions Inactive
@github-actions github-actions bot temporarily deployed to storybook-preview-2768 January 27, 2023 18:45 Inactive
…ing-enforce-what-as-can-be-set-to-with-a-runtime-validation-rule
@agreenberry agreenberry enabled auto-merge (squash) January 30, 2023 18:11
@agreenberry agreenberry temporarily deployed to github-pages January 30, 2023 18:18 — with GitHub Actions Inactive
@agreenberry agreenberry merged commit 5055b91 into main Jan 30, 2023
@agreenberry agreenberry deleted the 1572-accessibility-request-primer-react-heading-enforce-what-as-can-be-set-to-with-a-runtime-validation-rule branch January 30, 2023 18:22
@primer-css primer-css mentioned this pull request Jan 30, 2023
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