Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Token List: Add type checking#18839

Merged
aduth merged 2 commits into
masterfrom
update/token-list-types
Dec 7, 2019
Merged

Token List: Add type checking#18839
aduth merged 2 commits into
masterfrom
update/token-list-types

Conversation

@aduth

@aduth aduth commented Nov 30, 2019

Copy link
Copy Markdown
Member

Part of: #18838

This pull request seeks to enable type checking for the @wordpress/token-list package.

Testing Instructions:

Verify no regressions in modified behavior:

npm run test-unit

Verify types checking passes:

npm run lint:types

@aduth aduth added [Type] Automated Testing Testing infrastructure changes impacting the execution of end-to-end (E2E) and/or unit tests. [Type] Code Quality Issues or PRs that relate to code quality [Package] Token List /packages/token-list labels Nov 30, 2019

@gziolo gziolo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, not that much work to get all benefits 👍

Comment thread packages/token-list/src/index.js Outdated
@aduth aduth merged commit 2924efa into master Dec 7, 2019
@aduth aduth deleted the update/token-list-types branch December 7, 2019 21:13
@youknowriad youknowriad added this to the Gutenberg 7.1 milestone Dec 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Package] Token List /packages/token-list [Type] Automated Testing Testing infrastructure changes impacting the execution of end-to-end (E2E) and/or unit tests. [Type] Code Quality Issues or PRs that relate to code quality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants