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 comments from compiled styles #43177

Merged
merged 1 commit into from
Oct 24, 2022

Conversation

aristath
Copy link
Member

What?

Fixes #43173

@aristath aristath requested review from Mamaduka and gziolo August 12, 2022 10:12
@github-actions
Copy link

github-actions bot commented Aug 12, 2022

Size Change: -24.5 kB (-2%)

Total Size: 1.25 MB

Filename Size Change
build/block-directory/index.min.js 7.05 kB +469 B (+7%) 🔍
build/block-editor/index.min.js 162 kB +5.78 kB (+4%)
build/block-editor/style-rtl.css 15.2 kB +503 B (+3%)
build/block-editor/style.css 15.2 kB +509 B (+3%)
build/block-library/blocks/audio/style-rtl.css 122 B +19 B (+18%) ⚠️
build/block-library/blocks/audio/style.css 122 B +19 B (+18%) ⚠️
build/block-library/blocks/audio/theme-rtl.css 126 B +16 B (+15%) ⚠️
build/block-library/blocks/audio/theme.css 126 B +16 B (+15%) ⚠️
build/block-library/blocks/avatar/style-rtl.css 84 B +25 B (+42%) 🚨
build/block-library/blocks/avatar/style.css 84 B +25 B (+42%) 🚨
build/block-library/blocks/button/style-rtl.css 505 B -34 B (-6%)
build/block-library/blocks/button/style.css 505 B -34 B (-6%)
build/block-library/blocks/categories/style-rtl.css 100 B +21 B (+27%) 🚨
build/block-library/blocks/categories/style.css 100 B +21 B (+27%) 🚨
build/block-library/blocks/cover/editor-rtl.css 605 B -10 B (-2%)
build/block-library/blocks/cover/editor.css 607 B -9 B (-1%)
build/block-library/blocks/embed/theme-rtl.css 126 B +16 B (+15%) ⚠️
build/block-library/blocks/embed/theme.css 126 B +16 B (+15%) ⚠️
build/block-library/blocks/group/editor-rtl.css 337 B -2 B (-1%)
build/block-library/blocks/group/editor.css 337 B -2 B (-1%)
build/block-library/blocks/image/editor-rtl.css 876 B +140 B (+19%) ⚠️
build/block-library/blocks/image/editor.css 873 B +136 B (+18%) ⚠️
build/block-library/blocks/image/theme-rtl.css 126 B +16 B (+15%) ⚠️
build/block-library/blocks/image/theme.css 126 B +16 B (+15%) ⚠️
build/block-library/blocks/media-text/style-rtl.css 507 B +14 B (+3%)
build/block-library/blocks/media-text/style.css 505 B +15 B (+3%)
build/block-library/blocks/navigation/editor-rtl.css 1.99 kB -37 B (-2%)
build/block-library/blocks/navigation/editor.css 2 kB -41 B (-2%)
build/block-library/blocks/navigation/style-rtl.css 2.15 kB +179 B (+9%) 🔍
build/block-library/blocks/navigation/style.css 2.14 kB +173 B (+9%) 🔍
build/block-library/blocks/paragraph/editor-rtl.css 174 B +17 B (+11%) ⚠️
build/block-library/blocks/paragraph/editor.css 174 B +17 B (+11%) ⚠️
build/block-library/blocks/post-featured-image/editor-rtl.css 507 B -98 B (-16%) 👏
build/block-library/blocks/post-featured-image/editor.css 505 B -100 B (-17%) 👏
build/block-library/blocks/post-featured-image/style-rtl.css 166 B +13 B (+8%) 🔍
build/block-library/blocks/post-featured-image/style.css 166 B +13 B (+8%) 🔍
build/block-library/blocks/post-title/style-rtl.css 100 B +20 B (+25%) 🚨
build/block-library/blocks/post-title/style.css 100 B +20 B (+25%) 🚨
build/block-library/blocks/pullquote/editor-rtl.css 135 B -63 B (-32%) 🎉
build/block-library/blocks/pullquote/editor.css 135 B -63 B (-32%) 🎉
build/block-library/blocks/pullquote/style-rtl.css 326 B -44 B (-12%) 👏
build/block-library/blocks/pullquote/style.css 325 B -45 B (-12%) 👏
build/block-library/blocks/site-logo/editor-rtl.css 488 B -220 B (-31%) 🎉
build/block-library/blocks/site-logo/editor.css 488 B -220 B (-31%) 🎉
build/block-library/blocks/site-logo/style-rtl.css 203 B +11 B (+6%) 🔍
build/block-library/blocks/site-logo/style.css 203 B +11 B (+6%) 🔍
build/block-library/blocks/social-links/style-rtl.css 1.4 kB +10 B (+1%)
build/block-library/blocks/social-links/style.css 1.39 kB +10 B (+1%)
build/block-library/blocks/table/theme-rtl.css 190 B +15 B (+9%) 🔍
build/block-library/blocks/table/theme.css 190 B +15 B (+9%) 🔍
build/block-library/blocks/video/style-rtl.css 174 B +15 B (+9%) 🔍
build/block-library/blocks/video/style.css 174 B +15 B (+9%) 🔍
build/block-library/blocks/video/theme-rtl.css 126 B +16 B (+15%) ⚠️
build/block-library/blocks/video/theme.css 126 B +16 B (+15%) ⚠️
build/block-library/common-rtl.css 1.02 kB +12 B (+1%)
build/block-library/common.css 1.02 kB +12 B (+1%)
build/block-library/editor-rtl.css 11 kB +26 B (0%)
build/block-library/editor.css 11 kB +15 B (0%)
build/block-library/index.min.js 188 kB +2.91 kB (+2%)
build/block-library/style-rtl.css 12 kB +145 B (+1%)
build/block-library/style.css 12 kB +150 B (+1%)
build/block-library/theme-rtl.css 719 B +24 B (+3%)
build/block-library/theme.css 722 B +22 B (+3%)
build/block-serialization-default-parser/index.min.js 1.1 kB -12 B (-1%)
build/blocks/index.min.js 49.6 kB +2.23 kB (+5%) 🔍
build/components/index.min.js 198 kB -33.5 kB (-15%) 👏
build/components/style-rtl.css 11.5 kB -2.54 kB (-18%) 👏
build/components/style.css 11.5 kB -2.54 kB (-18%) 👏
build/compose/index.min.js 12 kB +9 B (0%)
build/core-data/index.min.js 15.5 kB +234 B (+2%)
build/customize-widgets/index.min.js 11.3 kB -3 B (0%)
build/customize-widgets/style-rtl.css 1.38 kB -24 B (-2%)
build/customize-widgets/style.css 1.38 kB -24 B (-2%)
build/data/index.min.js 8.06 kB +34 B (0%)
build/date/index.min.js 32.1 kB +68 B (0%)
build/dom/index.min.js 4.7 kB +7 B (0%)
build/edit-navigation/index.min.js 16 kB +11 B (0%)
build/edit-navigation/style-rtl.css 3.99 kB -28 B (-1%)
build/edit-navigation/style.css 4 kB -31 B (-1%)
build/edit-post/index.min.js 30.7 kB +171 B (+1%)
build/edit-post/style-rtl.css 6.94 kB -3 B (0%)
build/edit-post/style.css 6.94 kB -1 B (0%)
build/edit-site/index.min.js 58.2 kB +1.35 kB (+2%)
build/edit-site/style-rtl.css 8.23 kB +1 B (0%)
build/edit-site/style.css 8.22 kB -3 B (0%)
build/edit-widgets/index.min.js 16.5 kB +6 B (0%)
build/edit-widgets/style-rtl.css 4.34 kB -7 B (0%)
build/edit-widgets/style.css 4.34 kB -5 B (0%)
build/editor/index.min.js 41.6 kB +165 B (0%)
build/element/index.min.js 4.68 kB +6 B (0%)
build/format-library/index.min.js 6.75 kB -3 B (0%)
build/keycodes/index.min.js 1.81 kB +18 B (+1%)
build/nux/index.min.js 2.05 kB -3 B (0%)
build/rich-text/index.min.js 10.4 kB -745 B (-7%)
build/viewport/index.min.js 1.09 kB +11 B (+1%)
build/widgets/index.min.js 7.19 kB +1 B (0%)
build/widgets/style-rtl.css 1.18 kB +27 B (+2%)
build/widgets/style.css 1.19 kB +28 B (+2%)
ℹ️ View Unchanged
Filename Size
build/a11y/index.min.js 982 B
build/annotations/index.min.js 2.76 kB
build/api-fetch/index.min.js 2.26 kB
build/autop/index.min.js 2.14 kB
build/blob/index.min.js 475 B
build/block-directory/style-rtl.css 990 B
build/block-directory/style.css 991 B
build/block-editor/default-editor-styles-rtl.css 378 B
build/block-editor/default-editor-styles.css 378 B
build/block-library/blocks/archives/editor-rtl.css 61 B
build/block-library/blocks/archives/editor.css 60 B
build/block-library/blocks/archives/style-rtl.css 65 B
build/block-library/blocks/archives/style.css 65 B
build/block-library/blocks/audio/editor-rtl.css 150 B
build/block-library/blocks/audio/editor.css 150 B
build/block-library/blocks/avatar/editor-rtl.css 116 B
build/block-library/blocks/avatar/editor.css 116 B
build/block-library/blocks/block/editor-rtl.css 161 B
build/block-library/blocks/block/editor.css 161 B
build/block-library/blocks/button/editor-rtl.css 441 B
build/block-library/blocks/button/editor.css 441 B
build/block-library/blocks/buttons/editor-rtl.css 292 B
build/block-library/blocks/buttons/editor.css 292 B
build/block-library/blocks/buttons/style-rtl.css 275 B
build/block-library/blocks/buttons/style.css 275 B
build/block-library/blocks/calendar/style-rtl.css 207 B
build/block-library/blocks/calendar/style.css 207 B
build/block-library/blocks/categories/editor-rtl.css 84 B
build/block-library/blocks/categories/editor.css 83 B
build/block-library/blocks/code/editor-rtl.css 53 B
build/block-library/blocks/code/editor.css 53 B
build/block-library/blocks/code/style-rtl.css 103 B
build/block-library/blocks/code/style.css 103 B
build/block-library/blocks/code/theme-rtl.css 124 B
build/block-library/blocks/code/theme.css 124 B
build/block-library/blocks/columns/editor-rtl.css 108 B
build/block-library/blocks/columns/editor.css 108 B
build/block-library/blocks/columns/style-rtl.css 406 B
build/block-library/blocks/columns/style.css 406 B
build/block-library/blocks/comment-author-avatar/editor-rtl.css 125 B
build/block-library/blocks/comment-author-avatar/editor.css 125 B
build/block-library/blocks/comment-content/style-rtl.css 92 B
build/block-library/blocks/comment-content/style.css 92 B
build/block-library/blocks/comment-template/style-rtl.css 187 B
build/block-library/blocks/comment-template/style.css 185 B
build/block-library/blocks/comments-pagination-numbers/editor-rtl.css 123 B
build/block-library/blocks/comments-pagination-numbers/editor.css 121 B
build/block-library/blocks/comments-pagination/editor-rtl.css 222 B
build/block-library/blocks/comments-pagination/editor.css 209 B
build/block-library/blocks/comments-pagination/style-rtl.css 235 B
build/block-library/blocks/comments-pagination/style.css 231 B
build/block-library/blocks/comments-title/editor-rtl.css 75 B
build/block-library/blocks/comments-title/editor.css 75 B
build/block-library/blocks/comments/editor-rtl.css 834 B
build/block-library/blocks/comments/editor.css 832 B
build/block-library/blocks/comments/style-rtl.css 632 B
build/block-library/blocks/comments/style.css 630 B
build/block-library/blocks/cover/style-rtl.css 1.55 kB
build/block-library/blocks/cover/style.css 1.55 kB
build/block-library/blocks/embed/editor-rtl.css 293 B
build/block-library/blocks/embed/editor.css 293 B
build/block-library/blocks/embed/style-rtl.css 410 B
build/block-library/blocks/embed/style.css 410 B
build/block-library/blocks/file/editor-rtl.css 300 B
build/block-library/blocks/file/editor.css 300 B
build/block-library/blocks/file/style-rtl.css 253 B
build/block-library/blocks/file/style.css 254 B
build/block-library/blocks/file/view.min.js 346 B
build/block-library/blocks/freeform/editor-rtl.css 2.44 kB
build/block-library/blocks/freeform/editor.css 2.44 kB
build/block-library/blocks/gallery/editor-rtl.css 948 B
build/block-library/blocks/gallery/editor.css 950 B
build/block-library/blocks/gallery/style-rtl.css 1.53 kB
build/block-library/blocks/gallery/style.css 1.53 kB
build/block-library/blocks/gallery/theme-rtl.css 108 B
build/block-library/blocks/gallery/theme.css 108 B
build/block-library/blocks/group/style-rtl.css 57 B
build/block-library/blocks/group/style.css 57 B
build/block-library/blocks/group/theme-rtl.css 78 B
build/block-library/blocks/group/theme.css 78 B
build/block-library/blocks/heading/style-rtl.css 76 B
build/block-library/blocks/heading/style.css 76 B
build/block-library/blocks/html/editor-rtl.css 327 B
build/block-library/blocks/html/editor.css 329 B
build/block-library/blocks/image/style-rtl.css 627 B
build/block-library/blocks/image/style.css 630 B
build/block-library/blocks/latest-comments/style-rtl.css 284 B
build/block-library/blocks/latest-comments/style.css 284 B
build/block-library/blocks/latest-posts/editor-rtl.css 213 B
build/block-library/blocks/latest-posts/editor.css 212 B
build/block-library/blocks/latest-posts/style-rtl.css 463 B
build/block-library/blocks/latest-posts/style.css 462 B
build/block-library/blocks/list/style-rtl.css 88 B
build/block-library/blocks/list/style.css 88 B
build/block-library/blocks/media-text/editor-rtl.css 266 B
build/block-library/blocks/media-text/editor.css 263 B
build/block-library/blocks/more/editor-rtl.css 431 B
build/block-library/blocks/more/editor.css 431 B
build/block-library/blocks/navigation-link/editor-rtl.css 705 B
build/block-library/blocks/navigation-link/editor.css 703 B
build/block-library/blocks/navigation-link/style-rtl.css 115 B
build/block-library/blocks/navigation-link/style.css 115 B
build/block-library/blocks/navigation-submenu/editor-rtl.css 296 B
build/block-library/blocks/navigation-submenu/editor.css 295 B
build/block-library/blocks/navigation-submenu/view.min.js 423 B
build/block-library/blocks/navigation/view-modal.min.js 2.78 kB
build/block-library/blocks/navigation/view.min.js 443 B
build/block-library/blocks/nextpage/editor-rtl.css 395 B
build/block-library/blocks/nextpage/editor.css 395 B
build/block-library/blocks/page-list/editor-rtl.css 363 B
build/block-library/blocks/page-list/editor.css 363 B
build/block-library/blocks/page-list/style-rtl.css 175 B
build/block-library/blocks/page-list/style.css 175 B
build/block-library/blocks/paragraph/style-rtl.css 260 B
build/block-library/blocks/paragraph/style.css 260 B
build/block-library/blocks/post-author/style-rtl.css 175 B
build/block-library/blocks/post-author/style.css 176 B
build/block-library/blocks/post-comments-form/editor-rtl.css 96 B
build/block-library/blocks/post-comments-form/editor.css 96 B
build/block-library/blocks/post-comments-form/style-rtl.css 493 B
build/block-library/blocks/post-comments-form/style.css 493 B
build/block-library/blocks/post-date/style-rtl.css 61 B
build/block-library/blocks/post-date/style.css 61 B
build/block-library/blocks/post-excerpt/editor-rtl.css 73 B
build/block-library/blocks/post-excerpt/editor.css 73 B
build/block-library/blocks/post-excerpt/style-rtl.css 69 B
build/block-library/blocks/post-excerpt/style.css 69 B
build/block-library/blocks/post-template/editor-rtl.css 99 B
build/block-library/blocks/post-template/editor.css 98 B
build/block-library/blocks/post-template/style-rtl.css 282 B
build/block-library/blocks/post-template/style.css 282 B
build/block-library/blocks/post-terms/style-rtl.css 73 B
build/block-library/blocks/post-terms/style.css 73 B
build/block-library/blocks/preformatted/style-rtl.css 103 B
build/block-library/blocks/preformatted/style.css 103 B
build/block-library/blocks/pullquote/theme-rtl.css 167 B
build/block-library/blocks/pullquote/theme.css 167 B
build/block-library/blocks/query-pagination-numbers/editor-rtl.css 122 B
build/block-library/blocks/query-pagination-numbers/editor.css 121 B
build/block-library/blocks/query-pagination/editor-rtl.css 221 B
build/block-library/blocks/query-pagination/editor.css 211 B
build/block-library/blocks/query-pagination/style-rtl.css 282 B
build/block-library/blocks/query-pagination/style.css 278 B
build/block-library/blocks/query-title/style-rtl.css 63 B
build/block-library/blocks/query-title/style.css 63 B
build/block-library/blocks/query/editor-rtl.css 439 B
build/block-library/blocks/query/editor.css 439 B
build/block-library/blocks/quote/style-rtl.css 213 B
build/block-library/blocks/quote/style.css 213 B
build/block-library/blocks/quote/theme-rtl.css 223 B
build/block-library/blocks/quote/theme.css 226 B
build/block-library/blocks/read-more/style-rtl.css 132 B
build/block-library/blocks/read-more/style.css 132 B
build/block-library/blocks/rss/editor-rtl.css 202 B
build/block-library/blocks/rss/editor.css 204 B
build/block-library/blocks/rss/style-rtl.css 289 B
build/block-library/blocks/rss/style.css 288 B
build/block-library/blocks/search/editor-rtl.css 165 B
build/block-library/blocks/search/editor.css 165 B
build/block-library/blocks/search/style-rtl.css 396 B
build/block-library/blocks/search/style.css 393 B
build/block-library/blocks/search/theme-rtl.css 114 B
build/block-library/blocks/search/theme.css 114 B
build/block-library/blocks/separator/editor-rtl.css 146 B
build/block-library/blocks/separator/editor.css 146 B
build/block-library/blocks/separator/style-rtl.css 233 B
build/block-library/blocks/separator/style.css 233 B
build/block-library/blocks/separator/theme-rtl.css 194 B
build/block-library/blocks/separator/theme.css 194 B
build/block-library/blocks/shortcode/editor-rtl.css 464 B
build/block-library/blocks/shortcode/editor.css 464 B
build/block-library/blocks/site-tagline/editor-rtl.css 86 B
build/block-library/blocks/site-tagline/editor.css 86 B
build/block-library/blocks/site-title/editor-rtl.css 84 B
build/block-library/blocks/site-title/editor.css 84 B
build/block-library/blocks/social-link/editor-rtl.css 184 B
build/block-library/blocks/social-link/editor.css 184 B
build/block-library/blocks/social-links/editor-rtl.css 674 B
build/block-library/blocks/social-links/editor.css 673 B
build/block-library/blocks/spacer/editor-rtl.css 322 B
build/block-library/blocks/spacer/editor.css 322 B
build/block-library/blocks/spacer/style-rtl.css 48 B
build/block-library/blocks/spacer/style.css 48 B
build/block-library/blocks/table/editor-rtl.css 494 B
build/block-library/blocks/table/editor.css 494 B
build/block-library/blocks/table/style-rtl.css 611 B
build/block-library/blocks/table/style.css 609 B
build/block-library/blocks/tag-cloud/style-rtl.css 239 B
build/block-library/blocks/tag-cloud/style.css 239 B
build/block-library/blocks/template-part/editor-rtl.css 235 B
build/block-library/blocks/template-part/editor.css 235 B
build/block-library/blocks/template-part/theme-rtl.css 101 B
build/block-library/blocks/template-part/theme.css 101 B
build/block-library/blocks/text-columns/editor-rtl.css 95 B
build/block-library/blocks/text-columns/editor.css 95 B
build/block-library/blocks/text-columns/style-rtl.css 166 B
build/block-library/blocks/text-columns/style.css 166 B
build/block-library/blocks/verse/style-rtl.css 87 B
build/block-library/blocks/verse/style.css 87 B
build/block-library/blocks/video/editor-rtl.css 561 B
build/block-library/blocks/video/editor.css 563 B
build/block-library/editor-elements-rtl.css 75 B
build/block-library/editor-elements.css 75 B
build/block-library/elements-rtl.css 54 B
build/block-library/elements.css 54 B
build/block-library/reset-rtl.css 478 B
build/block-library/reset.css 478 B
build/block-serialization-spec-parser/index.min.js 2.83 kB
build/data-controls/index.min.js 653 B
build/deprecated/index.min.js 507 B
build/dom-ready/index.min.js 324 B
build/edit-post/classic-rtl.css 546 B
build/edit-post/classic.css 547 B
build/editor/style-rtl.css 3.66 kB
build/editor/style.css 3.65 kB
build/escape-html/index.min.js 537 B
build/format-library/style-rtl.css 571 B
build/format-library/style.css 571 B
build/hooks/index.min.js 1.64 kB
build/html-entities/index.min.js 448 B
build/i18n/index.min.js 3.77 kB
build/is-shallow-equal/index.min.js 527 B
build/keyboard-shortcuts/index.min.js 1.78 kB
build/list-reusable-blocks/index.min.js 1.74 kB
build/list-reusable-blocks/style-rtl.css 835 B
build/list-reusable-blocks/style.css 835 B
build/media-utils/index.min.js 2.93 kB
build/notices/index.min.js 953 B
build/nux/style-rtl.css 732 B
build/nux/style.css 728 B
build/plugins/index.min.js 1.94 kB
build/preferences-persistence/index.min.js 2.22 kB
build/preferences/index.min.js 1.3 kB
build/primitives/index.min.js 933 B
build/priority-queue/index.min.js 612 B
build/react-i18n/index.min.js 696 B
build/react-refresh-entry/index.min.js 8.44 kB
build/react-refresh-runtime/index.min.js 7.31 kB
build/redux-routine/index.min.js 2.74 kB
build/reusable-blocks/index.min.js 2.21 kB
build/reusable-blocks/style-rtl.css 256 B
build/reusable-blocks/style.css 256 B
build/server-side-render/index.min.js 1.61 kB
build/shortcode/index.min.js 1.53 kB
build/token-list/index.min.js 644 B
build/url/index.min.js 3.61 kB
build/vendors/react-dom.min.js 38.5 kB
build/vendors/react.min.js 4.34 kB
build/warning/index.min.js 268 B
build/wordcount/index.min.js 1.06 kB

compressed-size-action

@gziolo gziolo added the [Type] Build Tooling Issues or PRs related to build tooling label Aug 12, 2022
@Mamaduka
Copy link
Member

@aristath, should we close this in favor of https://core.trac.wordpress.org/ticket/56388?

@aristath
Copy link
Member Author

Sure, makes sense to commit the one in WP instead of this one.
Closing, thanks for the follow-up!

@aristath aristath closed this Aug 31, 2022
@aristath
Copy link
Member Author

aristath commented Sep 7, 2022

I'm reopening this one because tests are failing in the Core PR: WordPress/wordpress-develop#3095
From what I understand, this PR would first need to be merged in Gutenberg, then the one in Core can be merged once tests pass

@ndiego
Copy link
Member

ndiego commented Oct 10, 2022

Hi @aristath I am seeing this PR on the 6.1 project board. Does this need to be backported to 6.1, or are these Gutenberg only changed. I am a bit confused 😅 If the later, can I remove this from the 6.1 board?

@aristath
Copy link
Member Author

Hey @ndiego!
This needs to be merged in Gutenberg, so that the tests stop failing in the relevant Core PR. So this is basically a prerequisite for a commit that is supposed to go in 6.1 in Core 👍

Copy link
Contributor

@cbravobernal cbravobernal left a comment

Choose a reason for hiding this comment

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

LGTM

@cbravobernal cbravobernal merged commit a18d9ce into trunk Oct 24, 2022
@cbravobernal cbravobernal deleted the try/remove-compiled-styles-comments branch October 24, 2022 08:56
@github-actions github-actions bot added this to the Gutenberg 14.5 milestone Oct 24, 2022
@cbravobernal
Copy link
Contributor

Hi @aristath !
Should this PR be cherry-picked for RC3?

@aristath
Copy link
Member Author

aristath commented Oct 24, 2022

Thank you @c4rl0sbr4v0 !

Should this PR be cherry-picked for RC3?

This PR goes hand-in-hand with https://core.trac.wordpress.org/ticket/56388, so I'd say yes if we want to merge that PR in Core as well 👍

@cbravobernal cbravobernal added the Backport to WP 6.7 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Oct 24, 2022
@peterwilsoncc
Copy link
Contributor

Re backporting: I am not keen on changing the build setup like this so late in the cycle.

I'd feel differently if it affected *.min.css files but as it only affects users with SCRIPT_DEBUG: true in their wp-config.php file, I think it's safer to wait until 6.2

However, it's probably best to ask @jeffpaul @getsource and @dream-encode as the WP 6.1 tech leads.

@cbravobernal cbravobernal removed the Backport to WP 6.7 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Oct 25, 2022
@cbravobernal
Copy link
Contributor

Let's remove the label not to be auto-included then.

@hellofromtonya
Copy link
Contributor

Following up: Decision made today to punt the corresponding Core Trac ticket. Why?

See @peterwilsoncc's comments above and the explanation:

It's too late in the 6.1 cycle with RC3 happening in a few minutes and final release next week. Moving this ticket to 6.2 cycle.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Build Tooling Issues or PRs related to build tooling
Projects
No open projects
Status: Bumped to 6.1.1
Status: Bumped to 6.2
Development

Successfully merging this pull request may close these issues.

Compiled block styles contain a lot of trash
7 participants