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

Packages: Remove package import statements in favor of store object usage #27368

Merged
merged 1 commit into from
Nov 30, 2020

Conversation

gziolo
Copy link
Member

@gziolo gziolo commented Nov 30, 2020

Description

Related to #27088.

It's a general cleanup PR that remove package import statements in favor of store object usage for refactored stores.

Testing

Nothing particular specific on my mind here:
npm run lint-js
npm run test-unit
npm run test-e2e

You also might want to test manually whether Block Directory still works as expected. To do so, open a block inserter and type the name of the block that doesn't ship with Gutenberg core like: boxer. You should be able to install it and insert through the inserter.

@gziolo gziolo force-pushed the update/package-imports-stores branch from 1d68802 to b9d2265 Compare November 30, 2020 09:38
@gziolo gziolo self-assigned this Nov 30, 2020
@gziolo gziolo added npm Packages Related to npm packages [Type] Code Quality Issues or PRs that relate to code quality labels Nov 30, 2020
@github-actions
Copy link

github-actions bot commented Nov 30, 2020

Size Change: -86 B (0%)

Total Size: 1.19 MB

Filename Size Change
build/block-directory/index.js 8.73 kB +3 B (0%)
build/block-editor/index.js 128 kB -55 B (0%)
build/block-library/index.js 148 kB +7 B (0%)
build/components/index.js 172 kB -3 B (0%)
build/edit-navigation/index.js 11.1 kB -16 B (0%)
build/edit-post/index.js 306 kB -21 B (0%)
build/edit-site/index.js 24.1 kB -1 B
build/edit-widgets/index.js 26.3 kB -6 B (0%)
build/editor/index.js 43.3 kB +14 B (0%)
build/list-reusable-blocks/index.js 3.1 kB -3 B (0%)
build/media-utils/index.js 5.32 kB -1 B
build/plugins/index.js 2.56 kB -2 B (0%)
build/rich-text/index.js 13.4 kB -1 B
build/server-side-render/index.js 2.77 kB -1 B
ℹ️ View Unchanged
Filename Size Change
build/a11y/index.js 1.14 kB 0 B
build/annotations/index.js 3.8 kB 0 B
build/api-fetch/index.js 3.42 kB 0 B
build/autop/index.js 2.84 kB 0 B
build/blob/index.js 665 B 0 B
build/block-directory/style-rtl.css 943 B 0 B
build/block-directory/style.css 942 B 0 B
build/block-editor/style-rtl.css 11.2 kB 0 B
build/block-editor/style.css 11.2 kB 0 B
build/block-library/editor-rtl.css 8.95 kB 0 B
build/block-library/editor.css 8.95 kB 0 B
build/block-library/style-rtl.css 8.27 kB 0 B
build/block-library/style.css 8.27 kB 0 B
build/block-library/theme-rtl.css 789 B 0 B
build/block-library/theme.css 790 B 0 B
build/block-serialization-default-parser/index.js 1.87 kB 0 B
build/block-serialization-spec-parser/index.js 3.06 kB 0 B
build/blocks/index.js 48.1 kB 0 B
build/components/style-rtl.css 15.3 kB 0 B
build/components/style.css 15.3 kB 0 B
build/compose/index.js 9.95 kB 0 B
build/core-data/index.js 14.8 kB 0 B
build/data-controls/index.js 827 B 0 B
build/data/index.js 8.98 kB 0 B
build/date/index.js 11.2 kB 0 B
build/deprecated/index.js 769 B 0 B
build/dom-ready/index.js 571 B 0 B
build/dom/index.js 4.95 kB 0 B
build/edit-navigation/style-rtl.css 881 B 0 B
build/edit-navigation/style.css 885 B 0 B
build/edit-post/style-rtl.css 6.42 kB 0 B
build/edit-post/style.css 6.4 kB 0 B
build/edit-site/style-rtl.css 3.91 kB 0 B
build/edit-site/style.css 3.91 kB 0 B
build/edit-widgets/style-rtl.css 3.13 kB 0 B
build/edit-widgets/style.css 3.13 kB 0 B
build/editor/editor-styles-rtl.css 476 B 0 B
build/editor/editor-styles.css 478 B 0 B
build/editor/style-rtl.css 3.85 kB 0 B
build/editor/style.css 3.85 kB 0 B
build/element/index.js 4.63 kB 0 B
build/escape-html/index.js 735 B 0 B
build/format-library/index.js 6.86 kB 0 B
build/format-library/style-rtl.css 547 B 0 B
build/format-library/style.css 548 B 0 B
build/hooks/index.js 2.27 kB 0 B
build/html-entities/index.js 622 B 0 B
build/i18n/index.js 3.57 kB 0 B
build/is-shallow-equal/index.js 698 B 0 B
build/keyboard-shortcuts/index.js 2.54 kB 0 B
build/keycodes/index.js 1.94 kB 0 B
build/list-reusable-blocks/style-rtl.css 476 B 0 B
build/list-reusable-blocks/style.css 476 B 0 B
build/notices/index.js 1.82 kB 0 B
build/nux/index.js 3.42 kB 0 B
build/nux/style-rtl.css 671 B 0 B
build/nux/style.css 668 B 0 B
build/primitives/index.js 1.43 kB 0 B
build/priority-queue/index.js 790 B 0 B
build/redux-routine/index.js 2.84 kB 0 B
build/reusable-blocks/index.js 2.93 kB 0 B
build/shortcode/index.js 1.69 kB 0 B
build/token-list/index.js 1.27 kB 0 B
build/url/index.js 4.06 kB 0 B
build/viewport/index.js 1.86 kB 0 B
build/warning/index.js 1.14 kB 0 B
build/wordcount/index.js 1.22 kB 0 B

compressed-size-action

@gziolo gziolo force-pushed the update/package-imports-stores branch from b9d2265 to b69f991 Compare November 30, 2020 09:51
@gziolo gziolo merged commit 7b5d1a7 into master Nov 30, 2020
@gziolo gziolo deleted the update/package-imports-stores branch November 30, 2020 13:58
@github-actions github-actions bot added this to the Gutenberg 9.5 milestone Nov 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
npm Packages Related to npm packages [Type] Code Quality Issues or PRs that relate to code quality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant