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

Remove @wordpress/format-library dependency from edit-post #16034

Conversation

jorgefilipecosta
Copy link
Member

Description

The edit-post does not use the format-library package. The package is loaded by core and calls the correct mechanism to register the formats being used but the edit-post package does not directly depend on format-library.

How has this been tested?

I checked the formats still work as expected.

@jorgefilipecosta jorgefilipecosta added [Package] Edit Post /packages/edit-post [Type] Code Quality Issues or PRs that relate to code quality labels Jun 7, 2019
@jorgefilipecosta jorgefilipecosta merged commit 72badf2 into master Jun 7, 2019
@jorgefilipecosta jorgefilipecosta deleted the remove/wordpress-format-library-dependency-from-edit-post branch June 7, 2019 14:04
@aduth aduth added this to the Gutenberg 5.9 milestone Jun 7, 2019
@aduth
Copy link
Member

aduth commented Jun 7, 2019

This causes a local change to package-lock.json when running npm install. This should have been included with the changes.

Aside: Yes, this should be captured by build tooling for artifacts, and last it was considered, it seemed to be caused by a bug with npm ci.

@aduth
Copy link
Member

aduth commented Jun 7, 2019

I pushed the update direct to master in 60b667e.

aduth added a commit that referenced this pull request Jun 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Package] Edit Post /packages/edit-post [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