Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Blocks: Remove unused asType internal parser utility #16291

Merged
merged 1 commit into from
Jun 26, 2019

Conversation

aduth
Copy link
Member

@aduth aduth commented Jun 25, 2019

Previously: #10388, #10952

This pull request seeks to remove an unused asType function defined in the blocks parser. Previously, this was used to coerce attribute values to the type given by the attribute schema. This was later removed by a combination of #10388 and #10952, and the function is currently otherwise unused.

Testing Instructions:

Verify unit tests pass:

npm run test-unit

Ensure there are no lingering references to asType.

@aduth aduth added [Type] Code Quality Issues or PRs that relate to code quality [Package] Blocks /packages/blocks labels Jun 25, 2019
@aduth aduth requested a review from youknowriad June 25, 2019 20:06
@aduth aduth requested review from ellatrix and gziolo as code owners June 25, 2019 20:06
@aduth aduth merged commit c4fe20c into master Jun 26, 2019
@aduth aduth deleted the remove/blocks-as-type branch June 26, 2019 13:32
@github-actions github-actions bot added this to the Gutenberg 6.1 milestone Jun 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Package] Blocks /packages/blocks [Type] Code Quality Issues or PRs that relate to code quality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants