feat(project): add warning, invariant utilities and dev-expression plugin #2901

joshblack · 2023-02-14T18:06:59Z

This is a follow-up to the ADR: https://github.com/primer/react/blob/main/contributor-docs/adrs/adr-012-development-warnings.md

It adds several helpers:

warning for emitting warnings based on a condition
warn as an alternative to console.warn() that is removed in production builds
invariant which is used to assert states that must be true before moving on

The warning and invariant helpers are transformed by the added plugin: babel-plugin-deve-expression. This plugin will transform calls in the following way:

// Before
warning(condition, message);

// After
if (process.env.NODE_ENV !== 'production') {
  warning(condition, message);
}

changeset-bot · 2023-02-14T18:07:02Z

🦋 Changeset detected

Latest commit: efe6c0c

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 · 2023-02-14T18:10:25Z

size-limit report 📦

Path	Size
dist/browser.esm.js	94.48 KB (-0.22% 🔽)
dist/browser.umd.js	95.05 KB (-0.21% 🔽)

broccolinisoup

Looks great! Should we do a follow up PR to refactor the existing console.warns especially without the __DEV__ wraps?

…-to-project

joshblack · 2023-02-22T17:51:10Z

@broccolinisoup good call, I went ahead and opened up an ADR and included that as part of the impact of this kind of convention: https://github.com/primer/react/blob/321801aca743c519bd2dd40fe0f49503f2f60a26/contributor-docs/adrs/adr-XXX-development-warnings.md

If folks feel okay about this approach then I think you're right that we should go through and make sure calls to console.warn() are either wrapped in a __DEV__ block or use warning

…-to-project

joshblack · 2023-03-08T23:08:17Z

Re-requesting your review @broccolinisoup after that ADR was merged! Let me know if the changes make sense, I also added in an invariant() helper to help out with some of the DataTable stuff I'm working. Hope that's okay, let me know if not and I can remove!

broccolinisoup · 2023-03-09T08:00:16Z

Thanks @joshblack I'll review this tomorrow - I hope it is not super urgent!

src/utils/__tests__/warning.test.ts

broccolinisoup

Looks great ✨

…ugin (#2901) * chore(project): add warning utility and dev-expression plugin * chore: address eslint violations * feat: add invariant and warn helper * chore: flip flag for useMedia warning * chore: add changeset * chore: add minified exception case for invariant * chore: update type signature for invariant * test: update test for warning helper * refactor(hooks): update useControllableState warning usage * test(warning): update test titles with flipped condition --------- Co-authored-by: Josh Black <joshblack@users.noreply.github.com>

@deprecated

#2878) * Render ActionList.Group header inside list, use li for semantic HTML reasons. * Update snapshots. * Fix snapshot. No idea why this isn't being generated correctly. * Separate Heading from ActionList.Group. Also made some styling changes (ex. no vertical padding on ActionList <ul>) to align with PVC implementation. * removed usage of actionlist.group. * Fix type errors. * Create modern-coins-destroy.md * Fix ActionList.docs.json to fix components build CI step. * Update generated/components.json * test(vrt): update snapshots * test(vrt): update snapshots * Update snapshots. * Update themePreval snapshot and cache bust file. * Updated snapshots. * Revert padding on ActionList. * test(vrt): update snapshots * Make sure lists are surrounded by a div, fix accessibility issues in Storybook. * Updated snapshots, fix linting error. * Update docs. * Update generated/components.json * Fix linting issues. * Fixing some issues after the previous merge. * Fix ActionMenu feature stories. * Updated snapshots that shouldn't be different from main. * disable axe check * Update changeset to major patch instead of minor. * re-gen test * Add missing selectionVariant to Groups and Descriptions ActionMenu story. * test(vrt): update snapshots * Fix styling of ActionList to match prod. * Fix linting errors in ActionList. * test(vrt): update snapshots * Reduce padding between ActionLists in ActionMenu example. * refactor(DataTable): update optional type signatures (#2987) * refactor(DataTable): update optional type signatures * chore: update docs and add stories to json for DataTable * chore: remove default story * Update generated/components.json --------- Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * Textarea stories (#2988) * checkbox * Update generated/components.json * tests * Update generated/components.json * fix docs * radio stories * Update generated/components.json * fix doc * remove axe * add sb tests * select * add features * e2e * text input * Update generated/components.json * tokens * Update generated/components.json * tokens * textarea * all the things * Update generated/components.json * remove animation test * test(vrt): update snapshots * remove dead files * Update generated/components.json * missing exports --------- Co-authored-by: langermank <langermank@users.noreply.github.com> * Pagination design updates (#2702) * updates pagination styles to match latest token usage patterns, and updates ARIA markup * adds changeset * Create .changeset/long-bags-double.md --------- Co-authored-by: Cole Bemis <colebemis@github.com> * Make sure ActionList.Heading titles are bolded correctly. * Updated snapshots. * Revert title font weight to semibold. * Update snapshots. * test(vrt): update snapshots * Passing sxProps to ActionList.Heading through ActionList. * More snapshot fiddling. * Formatting. * Use margin and padding to get spacing correct. * Fix aat tests. * Update snapshots. * I think this should get the ActionMenu example back to what is expected. * Revert "test(vrt): update snapshots" This reverts commit a02012b. * Revert "test(vrt): update snapshots" This reverts commit a713875. * Revert "test(vrt): update snapshots" This reverts commit 2eae060. * Revert "test(vrt): update snapshots" This reverts commit bf59217. * Revert "test(vrt): update snapshots" This reverts commit 1c11a1b. * Revert "test(vrt): update snapshots" This reverts commit a02012b. * Revert "test(vrt): update snapshots" This reverts commit bf59217. * Fix underlinenav. * Updated snapshots. * test(vrt): update snapshots * Dialog v2: Move files, reformat stories and add e2e tests (#2984) * initial * Dialog v2: move files, refaormat stories and add e2e tests * Update generated/components.json * snaps * remove jest disable * Dialogv2 -> Dialog2 * storybook tests and await dialogs * disable animations * disable animations on the first snapshots too * fix page locators * test(vrt): update snapshots * test(Hidden): update export * Revert "test(Hidden): update export" This reverts commit f29edd9. --------- Co-authored-by: broccolinisoup <broccolinisoup@users.noreply.github.com> Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * feat(project): add warning, invariant utilities and dev-expression plugin (#2901) * chore(project): add warning utility and dev-expression plugin * chore: address eslint violations * feat: add invariant and warn helper * chore: flip flag for useMedia warning * chore: add changeset * chore: add minified exception case for invariant * chore: update type signature for invariant * test: update test for warning helper * refactor(hooks): update useControllableState warning usage * test(warning): update test titles with flipped condition --------- Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * chore(deps-dev): bump mdast-util-to-string from 3.1.0 to 3.1.1 (#3018) Bumps [mdast-util-to-string](https://github.com/syntax-tree/mdast-util-to-string) from 3.1.0 to 3.1.1. - [Release notes](https://github.com/syntax-tree/mdast-util-to-string/releases) - [Commits](syntax-tree/mdast-util-to-string@3.1.0...3.1.1) --- updated-dependencies: - dependency-name: mdast-util-to-string dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps-dev): bump jscodeshift from 0.13.0 to 0.14.0 (#3015) Bumps [jscodeshift](https://github.com/facebook/jscodeshift) from 0.13.0 to 0.14.0. - [Release notes](https://github.com/facebook/jscodeshift/releases) - [Changelog](https://github.com/facebook/jscodeshift/blob/main/CHANGELOG.md) - [Commits](facebook/jscodeshift@0.13.0...v0.14.0) --- updated-dependencies: - dependency-name: jscodeshift dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josh Black <joshblack@github.com> * chore(deps-dev): bump @babel/cli from 7.19.3 to 7.21.0 (#3016) Bumps [@babel/cli](https://github.com/babel/babel/tree/HEAD/packages/babel-cli) from 7.19.3 to 7.21.0. - [Release notes](https://github.com/babel/babel/releases) - [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md) - [Commits](https://github.com/babel/babel/commits/v7.21.0/packages/babel-cli) --- updated-dependencies: - dependency-name: "@babel/cli" dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josh Black <joshblack@github.com> * Add argTypes for HeadingPlayground ActionList story. * Add aria-busy declaration in relevant story instead of in component. * Update snapshots. * Recalculate autocomplete suggestions if the input data changes while the menu is open (#3009) * Add support for custom emoji suggestions * Allow declaratively setting suggestions as "loading" * Recalculate suggestions on update * Create .changeset/silly-plants-draw.md * Fix bug where page would be unresponsive if no suggestions provided * docs(DataTable): add reference stories for row actions (#2978) * docs(DataTable): add reference stories for row actions * chore: clean-up ts types * fix: add guards for optional id and field * docs: add story ids to DataTable docs json * Update generated/components.json --------- Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * chore(deps): update babel dependencies (#3005) Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * feat(DataTable): add support for alphanumeric, datatable, and custom sort functions (#3001) * chore: check-in work * chore: clean-up work * Update generated/components.json * docs: add custom sorting story --------- Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * Add new usage instructions to changeset. * Move ActionList.Group to versioned deprecated directory. * Fix references in deprecated ActionList.Group. * Fix themePreval snapshot. * Update src/ActionList/ActionList.stories.tsx Co-authored-by: Katie Langerman <18661030+langermank@users.noreply.github.com> * Update src/ActionList/ActionList.stories.tsx Co-authored-by: Katie Langerman <18661030+langermank@users.noreply.github.com> * Heading playground only uses heading props. * Use Box instead of styled elements for ActionList.Heading. * Remove presentation role and aria-hidden from ActionList.Heading. * Updated snapshot. * Use restricted as prop instead of headingLevel. * Use slots and contexts to allow developers to declare Heading like a child while still rendering correctly. * Linting. * Add @deprecated tag to ActionList.Group as first step towards deprecating component. * Fix type errors in ActionMenu and ActionList stories. * Formatting. * Updated snapshots. * Updating snapshots with --no-cache flag. * Update docs, fix missing Group export on ActionList. * Terminate comment correctly. * Regenerate ActionMenu e2e tests. * Add padding to fix snapshot diff. * Change padding again. * Remove extra padding. * Add small amount of margin (not padding) to fix snapshot diff. * Less margin. * More margin, actually. * I think this should work. * I think this is the correct margin, something else is off. * . * These snapshot comparisons are confusing me. * Just a little more margin. * Ugh. * test(vrt): update snapshots * Slightly less margin. * More tweaking. * More tweaking. * Disable animations on actionmenu test to see if it helps generate the snapshot correctly. * test(vrt): update snapshots * Don't animate Multiple Sections e2e test. * Format file. * test(vrt): update snapshots * Revert snapshots to match main. * Update snapshots again. * Build snapshots without cache. * Remove lingering references to headingProps. * Update generated/components.json * chore(deps): bump react-intersection-observer from 9.4.1 to 9.4.3 (#3177) Bumps [react-intersection-observer](https://github.com/thebuilder/react-intersection-observer) from 9.4.1 to 9.4.3. - [Release notes](https://github.com/thebuilder/react-intersection-observer/releases) - [Commits](thebuilder/react-intersection-observer@v9.4.1...v9.4.3) --- updated-dependencies: - dependency-name: react-intersection-observer dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps-dev): bump jest-fail-on-console from 3.0.2 to 3.1.1 (#3180) Bumps [jest-fail-on-console](https://github.com/ValentinH/jest-fail-on-console) from 3.0.2 to 3.1.1. - [Release notes](https://github.com/ValentinH/jest-fail-on-console/releases) - [Commits](ValentinH/jest-fail-on-console@v3.0.2...v3.1.1) --- updated-dependencies: - dependency-name: jest-fail-on-console dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps-dev): bump unist-util-find-before from 3.0.0 to 3.0.1 (#3179) Bumps [unist-util-find-before](https://github.com/syntax-tree/unist-util-find-before) from 3.0.0 to 3.0.1. - [Release notes](https://github.com/syntax-tree/unist-util-find-before/releases) - [Commits](syntax-tree/unist-util-find-before@3.0.0...3.0.1) --- updated-dependencies: - dependency-name: unist-util-find-before dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * fix(PageLayout): update Pane to warn instead of error (#3160) * fix(PageLayout): update Pane to warn instead of error * chore: add changeset * chore: remove invariant * chore: address eslint violations --------- Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * Refactor(PageLayout): use Playwright for interaction tests instead of Storybook. (#3171) * Change PageLayout interactions to use Playwright. * Build snapshots without cache. * Fix PageLayout.test.ts to use Page object because document is not defined. * Pass missing page parameter to isInViewPort function. * test(vrt): update snapshots * Add tabIndex to PullRequestPage test so it is a scrollable region. --------- Co-authored-by: radglob <radglob@users.noreply.github.com> * Delete docs migration-related code (#3187) * Delete docs_migration.yml * Delete docs-migration.js * Delete props-mdx-to-json.mjs * Docs(Tooltip): Adds `defaultValue` of tooltip direction to documentation (#3175) * docs: adds defaultValue of tooltip direction to documentation * remove the changeset * chore(project): update test-e2e docker image (#3183) * feat(project): add subpath pattern to package.json to restrict internal imports (#3186) * feat(project): add subpath pattern to package.json to restrict internal imports * refactor(project): move useOverflow to internal/hooks * docs(adrs): update adopted status for adr * chore: add changeset * Update generated/components.json --------- Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * Ensure disabled invisible buttons have the right text color. (#3185) * Set disabled invisible button color similarly to other button variants. * Don't set [data-no-visual=true] on invisible button variants. * Update snapshots. * Formatting and linting. * test(vrt): update snapshots * Update generated/components.json * chore(project): update test-e2e docker image (#3183) * Use selector to set disabled invisible button text color correctly. * test(vrt): update snapshots --------- Co-authored-by: radglob <radglob@users.noreply.github.com> Co-authored-by: Josh Black <joshblack@github.com> * Refactor(Overlay): move files to folder. (#3191) * Move overlay files to directory, move all stories to features as there isn't a clear default. * Update generated/components.json --------- Co-authored-by: radglob <radglob@users.noreply.github.com> * Refactor(Text): move files to directory, add storybook and e2e tests. (#3189) * Move Text files into directory, create storybook and e2e tests. * Fix Text reference in src/index.ts, update snapshots. * Update generated/components.json * test(vrt): update snapshots --------- Co-authored-by: radglob <radglob@users.noreply.github.com> * Reduce ❌ for forks (#2283) * Limit deploy to has pages * Limit statuses to primer/react * Limit stale to primer/react * Preview cross org forks --------- Co-authored-by: Mike Perrotti <mperrotti@github.com> Co-authored-by: Rez <rezrah@github.com> Co-authored-by: Josh Black <joshblack@github.com> * Setup snapshots for visual regression for experimental/Button2 (#3181) * duplicate button into drafts * update story title * add visual tests for Button2 * test(vrt): update snapshots --------- Co-authored-by: siddharthkp <siddharthkp@users.noreply.github.com> * Version Packages (#3106) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * Bug fix: `ButtonGroup` borders (#3198) * fix hover borders * Create wicked-knives-sparkle.md * snaps * why so many snaps? * revert? * test(vrt): update snapshots --------- Co-authored-by: langermank <langermank@users.noreply.github.com> * Refactor(Portal): move stories into Portal directory. (#3188) * Move portal stories to portal directory. * Reorganize portal stories into separate files for default and features. * Generate e2e tests for portal components. * Update generated/components.json * Remove e2e portal tests as they do not play well with Playwright. * Remove unnecessary decorator from story metadata. --------- Co-authored-by: radglob <radglob@users.noreply.github.com> * Use SSR-compatible slot implementation in ActionList/NavList (#3173) * Migrate ActionList to use new slots impl * Remove ActionList slots * Remove unused import * Create lemon-berries-run.md * Update snapshots * Fix slot description logic * Fix style logic * Fix conditional box logic * Fix aria-labelledby logic * Fix flexbox logic --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: radglob <radglob@users.noreply.github.com> Co-authored-by: langermank <18661030+langermank@users.noreply.github.com> Co-authored-by: joshblack <joshblack@users.noreply.github.com> Co-authored-by: Josh Black <joshblack@github.com> Co-authored-by: langermank <langermank@users.noreply.github.com> Co-authored-by: Mike Perrotti <mperrotti@github.com> Co-authored-by: Cole Bemis <colebemis@github.com> Co-authored-by: Armağan <broccolinisoup@github.com> Co-authored-by: broccolinisoup <broccolinisoup@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Ian Sanders <iansan5653@github.com> Co-authored-by: Ederson Lucas <27015559+edersonlucas@users.noreply.github.com> Co-authored-by: Josh Soref <2119212+jsoref@users.noreply.github.com> Co-authored-by: Rez <rezrah@github.com> Co-authored-by: Siddharth Kshetrapal <siddharthkp@github.com> Co-authored-by: siddharthkp <siddharthkp@users.noreply.github.com> Co-authored-by: GitHub Design Systems Bot <30705008+primer-css@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

@deprecated

#2878) * Render ActionList.Group header inside list, use li for semantic HTML reasons. * Update snapshots. * Fix snapshot. No idea why this isn't being generated correctly. * Separate Heading from ActionList.Group. Also made some styling changes (ex. no vertical padding on ActionList <ul>) to align with PVC implementation. * removed usage of actionlist.group. * Fix type errors. * Create modern-coins-destroy.md * Fix ActionList.docs.json to fix components build CI step. * Update generated/components.json * test(vrt): update snapshots * test(vrt): update snapshots * Update snapshots. * Update themePreval snapshot and cache bust file. * Updated snapshots. * Revert padding on ActionList. * test(vrt): update snapshots * Make sure lists are surrounded by a div, fix accessibility issues in Storybook. * Updated snapshots, fix linting error. * Update docs. * Update generated/components.json * Fix linting issues. * Fixing some issues after the previous merge. * Fix ActionMenu feature stories. * Updated snapshots that shouldn't be different from main. * disable axe check * Update changeset to major patch instead of minor. * re-gen test * Add missing selectionVariant to Groups and Descriptions ActionMenu story. * test(vrt): update snapshots * Fix styling of ActionList to match prod. * Fix linting errors in ActionList. * test(vrt): update snapshots * Reduce padding between ActionLists in ActionMenu example. * refactor(DataTable): update optional type signatures (#2987) * refactor(DataTable): update optional type signatures * chore: update docs and add stories to json for DataTable * chore: remove default story * Update generated/components.json --------- Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * Textarea stories (#2988) * checkbox * Update generated/components.json * tests * Update generated/components.json * fix docs * radio stories * Update generated/components.json * fix doc * remove axe * add sb tests * select * add features * e2e * text input * Update generated/components.json * tokens * Update generated/components.json * tokens * textarea * all the things * Update generated/components.json * remove animation test * test(vrt): update snapshots * remove dead files * Update generated/components.json * missing exports --------- Co-authored-by: langermank <langermank@users.noreply.github.com> * Pagination design updates (#2702) * updates pagination styles to match latest token usage patterns, and updates ARIA markup * adds changeset * Create .changeset/long-bags-double.md --------- Co-authored-by: Cole Bemis <colebemis@github.com> * Make sure ActionList.Heading titles are bolded correctly. * Updated snapshots. * Revert title font weight to semibold. * Update snapshots. * test(vrt): update snapshots * Passing sxProps to ActionList.Heading through ActionList. * More snapshot fiddling. * Formatting. * Use margin and padding to get spacing correct. * Fix aat tests. * Update snapshots. * I think this should get the ActionMenu example back to what is expected. * Revert "test(vrt): update snapshots" This reverts commit a02012b. * Revert "test(vrt): update snapshots" This reverts commit a713875. * Revert "test(vrt): update snapshots" This reverts commit 2eae060. * Revert "test(vrt): update snapshots" This reverts commit bf59217. * Revert "test(vrt): update snapshots" This reverts commit 1c11a1b. * Revert "test(vrt): update snapshots" This reverts commit a02012b. * Revert "test(vrt): update snapshots" This reverts commit bf59217. * Fix underlinenav. * Updated snapshots. * test(vrt): update snapshots * Dialog v2: Move files, reformat stories and add e2e tests (#2984) * initial * Dialog v2: move files, refaormat stories and add e2e tests * Update generated/components.json * snaps * remove jest disable * Dialogv2 -> Dialog2 * storybook tests and await dialogs * disable animations * disable animations on the first snapshots too * fix page locators * test(vrt): update snapshots * test(Hidden): update export * Revert "test(Hidden): update export" This reverts commit f29edd9. --------- Co-authored-by: broccolinisoup <broccolinisoup@users.noreply.github.com> Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * feat(project): add warning, invariant utilities and dev-expression plugin (#2901) * chore(project): add warning utility and dev-expression plugin * chore: address eslint violations * feat: add invariant and warn helper * chore: flip flag for useMedia warning * chore: add changeset * chore: add minified exception case for invariant * chore: update type signature for invariant * test: update test for warning helper * refactor(hooks): update useControllableState warning usage * test(warning): update test titles with flipped condition --------- Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * chore(deps-dev): bump mdast-util-to-string from 3.1.0 to 3.1.1 (#3018) Bumps [mdast-util-to-string](https://github.com/syntax-tree/mdast-util-to-string) from 3.1.0 to 3.1.1. - [Release notes](https://github.com/syntax-tree/mdast-util-to-string/releases) - [Commits](syntax-tree/mdast-util-to-string@3.1.0...3.1.1) --- updated-dependencies: - dependency-name: mdast-util-to-string dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps-dev): bump jscodeshift from 0.13.0 to 0.14.0 (#3015) Bumps [jscodeshift](https://github.com/facebook/jscodeshift) from 0.13.0 to 0.14.0. - [Release notes](https://github.com/facebook/jscodeshift/releases) - [Changelog](https://github.com/facebook/jscodeshift/blob/main/CHANGELOG.md) - [Commits](facebook/jscodeshift@0.13.0...v0.14.0) --- updated-dependencies: - dependency-name: jscodeshift dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josh Black <joshblack@github.com> * chore(deps-dev): bump @babel/cli from 7.19.3 to 7.21.0 (#3016) Bumps [@babel/cli](https://github.com/babel/babel/tree/HEAD/packages/babel-cli) from 7.19.3 to 7.21.0. - [Release notes](https://github.com/babel/babel/releases) - [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md) - [Commits](https://github.com/babel/babel/commits/v7.21.0/packages/babel-cli) --- updated-dependencies: - dependency-name: "@babel/cli" dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josh Black <joshblack@github.com> * Add argTypes for HeadingPlayground ActionList story. * Add aria-busy declaration in relevant story instead of in component. * Update snapshots. * Recalculate autocomplete suggestions if the input data changes while the menu is open (#3009) * Add support for custom emoji suggestions * Allow declaratively setting suggestions as "loading" * Recalculate suggestions on update * Create .changeset/silly-plants-draw.md * Fix bug where page would be unresponsive if no suggestions provided * docs(DataTable): add reference stories for row actions (#2978) * docs(DataTable): add reference stories for row actions * chore: clean-up ts types * fix: add guards for optional id and field * docs: add story ids to DataTable docs json * Update generated/components.json --------- Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * chore(deps): update babel dependencies (#3005) Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * feat(DataTable): add support for alphanumeric, datatable, and custom sort functions (#3001) * chore: check-in work * chore: clean-up work * Update generated/components.json * docs: add custom sorting story --------- Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * Add new usage instructions to changeset. * Move ActionList.Group to versioned deprecated directory. * Fix references in deprecated ActionList.Group. * Fix themePreval snapshot. * Update src/ActionList/ActionList.stories.tsx Co-authored-by: Katie Langerman <18661030+langermank@users.noreply.github.com> * Update src/ActionList/ActionList.stories.tsx Co-authored-by: Katie Langerman <18661030+langermank@users.noreply.github.com> * Heading playground only uses heading props. * Use Box instead of styled elements for ActionList.Heading. * Remove presentation role and aria-hidden from ActionList.Heading. * Updated snapshot. * Use restricted as prop instead of headingLevel. * Use slots and contexts to allow developers to declare Heading like a child while still rendering correctly. * Linting. * Add @deprecated tag to ActionList.Group as first step towards deprecating component. * Fix type errors in ActionMenu and ActionList stories. * Formatting. * Updated snapshots. * Updating snapshots with --no-cache flag. * Update docs, fix missing Group export on ActionList. * Terminate comment correctly. * Regenerate ActionMenu e2e tests. * Add padding to fix snapshot diff. * Change padding again. * Remove extra padding. * Add small amount of margin (not padding) to fix snapshot diff. * Less margin. * More margin, actually. * I think this should work. * I think this is the correct margin, something else is off. * . * These snapshot comparisons are confusing me. * Just a little more margin. * Ugh. * test(vrt): update snapshots * Slightly less margin. * More tweaking. * More tweaking. * Disable animations on actionmenu test to see if it helps generate the snapshot correctly. * test(vrt): update snapshots * Don't animate Multiple Sections e2e test. * Format file. * test(vrt): update snapshots * Revert snapshots to match main. * Update snapshots again. * Build snapshots without cache. * Remove lingering references to headingProps. * Update generated/components.json * chore(deps): bump react-intersection-observer from 9.4.1 to 9.4.3 (#3177) Bumps [react-intersection-observer](https://github.com/thebuilder/react-intersection-observer) from 9.4.1 to 9.4.3. - [Release notes](https://github.com/thebuilder/react-intersection-observer/releases) - [Commits](thebuilder/react-intersection-observer@v9.4.1...v9.4.3) --- updated-dependencies: - dependency-name: react-intersection-observer dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps-dev): bump jest-fail-on-console from 3.0.2 to 3.1.1 (#3180) Bumps [jest-fail-on-console](https://github.com/ValentinH/jest-fail-on-console) from 3.0.2 to 3.1.1. - [Release notes](https://github.com/ValentinH/jest-fail-on-console/releases) - [Commits](ValentinH/jest-fail-on-console@v3.0.2...v3.1.1) --- updated-dependencies: - dependency-name: jest-fail-on-console dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps-dev): bump unist-util-find-before from 3.0.0 to 3.0.1 (#3179) Bumps [unist-util-find-before](https://github.com/syntax-tree/unist-util-find-before) from 3.0.0 to 3.0.1. - [Release notes](https://github.com/syntax-tree/unist-util-find-before/releases) - [Commits](syntax-tree/unist-util-find-before@3.0.0...3.0.1) --- updated-dependencies: - dependency-name: unist-util-find-before dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * fix(PageLayout): update Pane to warn instead of error (#3160) * fix(PageLayout): update Pane to warn instead of error * chore: add changeset * chore: remove invariant * chore: address eslint violations --------- Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * Refactor(PageLayout): use Playwright for interaction tests instead of Storybook. (#3171) * Change PageLayout interactions to use Playwright. * Build snapshots without cache. * Fix PageLayout.test.ts to use Page object because document is not defined. * Pass missing page parameter to isInViewPort function. * test(vrt): update snapshots * Add tabIndex to PullRequestPage test so it is a scrollable region. --------- Co-authored-by: radglob <radglob@users.noreply.github.com> * Delete docs migration-related code (#3187) * Delete docs_migration.yml * Delete docs-migration.js * Delete props-mdx-to-json.mjs * Docs(Tooltip): Adds `defaultValue` of tooltip direction to documentation (#3175) * docs: adds defaultValue of tooltip direction to documentation * remove the changeset * chore(project): update test-e2e docker image (#3183) * feat(project): add subpath pattern to package.json to restrict internal imports (#3186) * feat(project): add subpath pattern to package.json to restrict internal imports * refactor(project): move useOverflow to internal/hooks * docs(adrs): update adopted status for adr * chore: add changeset * Update generated/components.json --------- Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * Ensure disabled invisible buttons have the right text color. (#3185) * Set disabled invisible button color similarly to other button variants. * Don't set [data-no-visual=true] on invisible button variants. * Update snapshots. * Formatting and linting. * test(vrt): update snapshots * Update generated/components.json * chore(project): update test-e2e docker image (#3183) * Use selector to set disabled invisible button text color correctly. * test(vrt): update snapshots --------- Co-authored-by: radglob <radglob@users.noreply.github.com> Co-authored-by: Josh Black <joshblack@github.com> * Refactor(Overlay): move files to folder. (#3191) * Move overlay files to directory, move all stories to features as there isn't a clear default. * Update generated/components.json --------- Co-authored-by: radglob <radglob@users.noreply.github.com> * Refactor(Text): move files to directory, add storybook and e2e tests. (#3189) * Move Text files into directory, create storybook and e2e tests. * Fix Text reference in src/index.ts, update snapshots. * Update generated/components.json * test(vrt): update snapshots --------- Co-authored-by: radglob <radglob@users.noreply.github.com> * Reduce ❌ for forks (#2283) * Limit deploy to has pages * Limit statuses to primer/react * Limit stale to primer/react * Preview cross org forks --------- Co-authored-by: Mike Perrotti <mperrotti@github.com> Co-authored-by: Rez <rezrah@github.com> Co-authored-by: Josh Black <joshblack@github.com> * Setup snapshots for visual regression for experimental/Button2 (#3181) * duplicate button into drafts * update story title * add visual tests for Button2 * test(vrt): update snapshots --------- Co-authored-by: siddharthkp <siddharthkp@users.noreply.github.com> * Version Packages (#3106) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * Bug fix: `ButtonGroup` borders (#3198) * fix hover borders * Create wicked-knives-sparkle.md * snaps * why so many snaps? * revert? * test(vrt): update snapshots --------- Co-authored-by: langermank <langermank@users.noreply.github.com> * Refactor(Portal): move stories into Portal directory. (#3188) * Move portal stories to portal directory. * Reorganize portal stories into separate files for default and features. * Generate e2e tests for portal components. * Update generated/components.json * Remove e2e portal tests as they do not play well with Playwright. * Remove unnecessary decorator from story metadata. --------- Co-authored-by: radglob <radglob@users.noreply.github.com> * Use SSR-compatible slot implementation in ActionList/NavList (#3173) * Migrate ActionList to use new slots impl * Remove ActionList slots * Remove unused import * Create lemon-berries-run.md * Update snapshots * Fix slot description logic * Fix style logic * Fix conditional box logic * Fix aria-labelledby logic * Fix flexbox logic --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: radglob <radglob@users.noreply.github.com> Co-authored-by: langermank <18661030+langermank@users.noreply.github.com> Co-authored-by: joshblack <joshblack@users.noreply.github.com> Co-authored-by: Josh Black <joshblack@github.com> Co-authored-by: langermank <langermank@users.noreply.github.com> Co-authored-by: Mike Perrotti <mperrotti@github.com> Co-authored-by: Cole Bemis <colebemis@github.com> Co-authored-by: Armağan <broccolinisoup@github.com> Co-authored-by: broccolinisoup <broccolinisoup@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Ian Sanders <iansan5653@github.com> Co-authored-by: Ederson Lucas <27015559+edersonlucas@users.noreply.github.com> Co-authored-by: Josh Soref <2119212+jsoref@users.noreply.github.com> Co-authored-by: Rez <rezrah@github.com> Co-authored-by: Siddharth Kshetrapal <siddharthkp@github.com> Co-authored-by: siddharthkp <siddharthkp@users.noreply.github.com> Co-authored-by: GitHub Design Systems Bot <30705008+primer-css@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

chore(project): add warning utility and dev-expression plugin

d9cb2f0

joshblack requested review from a team and mperrotti February 14, 2023 18:06

joshblack added the skip changeset This change does not need a changelog label Feb 14, 2023

joshblack mentioned this pull request Feb 14, 2023

PageHeader: A11y eng review remediations #2900

Merged

6 tasks

joshblack temporarily deployed to github-pages February 14, 2023 18:13 — with GitHub Actions Inactive

github-actions bot temporarily deployed to storybook-preview-2901 February 14, 2023 18:14 Inactive

broccolinisoup approved these changes Feb 15, 2023

View reviewed changes

joshblack added 2 commits February 22, 2023 11:31

Merge branch 'main' of github.com:primer/react into chore/add-warning…

2218347

…-to-project

chore: address eslint violations

c5289e9

joshblack temporarily deployed to github-pages February 22, 2023 17:42 — with GitHub Actions Inactive

github-actions bot temporarily deployed to storybook-preview-2901 February 22, 2023 17:42 Inactive

joshblack mentioned this pull request Feb 22, 2023

docs(adr): add development-warnings adr #2928

Merged

joshblack added 2 commits March 8, 2023 16:43

Merge branch 'main' of github.com:primer/react into chore/add-warning…

39d859c

…-to-project

feat: add invariant and warn helper

3a5c6d2

joshblack temporarily deployed to github-pages March 8, 2023 23:01 — with GitHub Actions Inactive

github-actions bot temporarily deployed to storybook-preview-2901 March 8, 2023 23:02 Inactive

joshblack changed the title ~~chore(project): add warning utility and dev-expression plugin~~ chore(project): add warning, invariant utilities and dev-expression plugin Mar 8, 2023

joshblack added 2 commits March 8, 2023 17:03

chore: flip flag for useMedia warning

6a548ed

chore: add changeset

b975ab2

joshblack removed the skip changeset This change does not need a changelog label Mar 8, 2023

joshblack enabled auto-merge March 8, 2023 23:04

joshblack disabled auto-merge March 8, 2023 23:04

joshblack added 2 commits March 8, 2023 17:06

chore: add minified exception case for invariant

062ea8a

chore: update type signature for invariant

585992d

joshblack changed the title ~~chore(project): add warning, invariant utilities and dev-expression plugin~~ feat(project): add warning, invariant utilities and dev-expression plugin Mar 8, 2023

joshblack requested a review from broccolinisoup March 8, 2023 23:07

github-actions bot temporarily deployed to storybook-preview-2901 March 8, 2023 23:10 Inactive

github-actions bot temporarily deployed to storybook-preview-2901 March 8, 2023 23:12 Inactive

joshblack temporarily deployed to github-pages March 8, 2023 23:13 — with GitHub Actions Inactive

github-actions bot temporarily deployed to storybook-preview-2901 March 8, 2023 23:13 Inactive

test: update test for warning helper

4dcb508

joshblack temporarily deployed to github-pages March 9, 2023 20:12 — with GitHub Actions Inactive

github-actions bot temporarily deployed to storybook-preview-2901 March 9, 2023 20:12 Inactive

refactor(hooks): update useControllableState warning usage

b51b7dd

joshblack temporarily deployed to github-pages March 9, 2023 20:45 — with GitHub Actions Inactive

github-actions bot temporarily deployed to storybook-preview-2901 March 9, 2023 20:46 Inactive

broccolinisoup reviewed Mar 10, 2023

View reviewed changes

src/utils/__tests__/warning.test.ts Outdated Show resolved Hide resolved

test(warning): update test titles with flipped condition

efe6c0c

joshblack requested a review from broccolinisoup March 10, 2023 16:39

joshblack deployed to github-pages March 10, 2023 16:44 — with GitHub Actions View deployment

github-actions bot deployed to storybook-preview-2901 March 10, 2023 16:45 View deployment

broccolinisoup approved these changes Mar 13, 2023

View reviewed changes

joshblack added this pull request to the merge queue Mar 13, 2023

Merged via the queue into main with commit 992f1ac Mar 13, 2023

joshblack deleted the chore/add-warning-to-project branch March 13, 2023 15:08

primer-css mentioned this pull request Mar 13, 2023

Release tracking #3021

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(project): add warning, invariant utilities and dev-expression plugin #2901

feat(project): add warning, invariant utilities and dev-expression plugin #2901

joshblack commented Feb 14, 2023 •

edited

Loading

changeset-bot bot commented Feb 14, 2023 •

edited

Loading

github-actions bot commented Feb 14, 2023 •

edited

Loading

broccolinisoup left a comment

joshblack commented Feb 22, 2023

joshblack commented Mar 8, 2023

broccolinisoup commented Mar 9, 2023

broccolinisoup left a comment

feat(project): add warning, invariant utilities and dev-expression plugin #2901

feat(project): add warning, invariant utilities and dev-expression plugin #2901

Conversation

joshblack commented Feb 14, 2023 • edited Loading

changeset-bot bot commented Feb 14, 2023 • edited Loading

🦋 Changeset detected

github-actions bot commented Feb 14, 2023 • edited Loading

size-limit report 📦

broccolinisoup left a comment

Choose a reason for hiding this comment

joshblack commented Feb 22, 2023

joshblack commented Mar 8, 2023

broccolinisoup commented Mar 9, 2023

broccolinisoup left a comment

Choose a reason for hiding this comment

joshblack commented Feb 14, 2023 •

edited

Loading

changeset-bot bot commented Feb 14, 2023 •

edited

Loading

github-actions bot commented Feb 14, 2023 •

edited

Loading