Last active
January 20, 2026 16:01
-
-
Save jwmatthews/394c34ac7280b1f0f21f4f487f86e80e to your computer and use it in GitHub Desktop.
2026 Jan 19 - Running pf-codemods
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| /Users/jmatthews/scratch/2026/01_19_patternfly_5_to_6/todds_app/patternfly-migration-workshop/src/App.tsx | |
| 48:9 error The values for the `align` property on ToolbarGroup have been updated from "alignLeft" and "alignRight" to "alignStart" and "alignEnd", respectively @patternfly/pf-codemods/toolbar-update-align-values | |
| 48:9 error The spacer property has been removed from ToolbarGroup. We recommend instead using our new gap, columnGap, or rowGap properties @patternfly/pf-codemods/toolbar-replaced-spacer-spaceItems | |
| 48:9 error The `icon-button-group` variant of ToolbarGroup has been renamed to `action-group-plain` @patternfly/pf-codemods/toolbarGroup-updated-variant | |
| 65:7 error The structure of Masthead has been updated, MastheadToggle should now be wrapped in MastheadMain @patternfly/pf-codemods/masthead-structure-changes | |
| 75:9 error The structure of Masthead has been updated, the PF5 MastheadBrand has been renamed to MastheadLogo (this renaming is handled by our masthead-name-changes codemod) and should now be wrapped in a new MastheadBrand @patternfly/pf-codemods/masthead-structure-changes | |
| 75:9 error MastheadBrand has been renamed to MastheadLogo @patternfly/pf-codemods/masthead-name-changes | |
| 175:5 error We've renamed the `header` prop for Page to `masthead` @patternfly/pf-codemods/page-rename-header | |
| /Users/jmatthews/scratch/2026/01_19_patternfly_5_to_6/todds_app/patternfly-migration-workshop/src/components/tier1-simple/StatusBadge.tsx | |
| 2:1 error Chip has been deprecated. Running the fix flag will update your imports to our deprecated package, but we suggest using Label instead @patternfly/pf-codemods/chip-deprecated | |
| /Users/jmatthews/scratch/2026/01_19_patternfly_5_to_6/todds_app/patternfly-migration-workshop/src/components/tier1-simple/UserProfile.tsx | |
| 2:1 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| 19:9 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| 20:11 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| 20:38 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| 21:11 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| 21:37 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| 22:11 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| 22:42 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| 23:9 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| /Users/jmatthews/scratch/2026/01_19_patternfly_5_to_6/todds_app/patternfly-migration-workshop/src/components/tier2-moderate/ActionMenu.tsx | |
| 2:1 warning We now recommend passing any icon to the `icon` prop instead of passing it as children, such as for plain, icon only toggles. Passing an icon as children will result in incorrect styling @patternfly/pf-codemods/menuToggle-warn-iconOnly-toggle | |
| 3:3 warning The default value of appendTo on Dropdown has been updated to `document.body` @patternfly/pf-codemods/popper-update-appendTo-default | |
| 2:1 error Unused PatternFly import DropdownToggle from '@patternfly/react-core' should be removed @patternfly/pf-codemods/no-unused-imports-v6 | |
| /Users/jmatthews/scratch/2026/01_19_patternfly_5_to_6/todds_app/patternfly-migration-workshop/src/components/tier2-moderate/EmptyStateExample.tsx | |
| 4:3 error EmptyStateIcon is no longer exported by PatternFly. This rule will not fix any imports, as our cleanup rule handles removal of unused imports @patternfly/pf-codemods/emptyState-nonExported-components | |
| 4:3 error EmptyStateHeader is no longer exported by PatternFly. This rule will not fix any imports, as our cleanup rule handles removal of unused imports @patternfly/pf-codemods/emptyState-nonExported-components | |
| 6:3 error EmptyStateIcon is no longer exported by PatternFly. This rule will not fix any imports, as our cleanup rule handles removal of unused imports @patternfly/pf-codemods/emptyState-nonExported-components | |
| 6:3 error EmptyStateHeader is no longer exported by PatternFly. This rule will not fix any imports, as our cleanup rule handles removal of unused imports @patternfly/pf-codemods/emptyState-nonExported-components | |
| 28:7 error EmptyStateHeader has been moved inside of the EmptyState component and is now only customizable using props. Additionally, the EmptyStateIcon component now wraps content passed to the icon prop automatically @patternfly/pf-codemods/emptyStateHeader-move-into-emptyState | |
| 42:5 error EmptyStateHeader has been moved inside of the EmptyState component and is now only customizable using props. Additionally, the EmptyStateIcon component now wraps content passed to the icon prop automatically @patternfly/pf-codemods/emptyStateHeader-move-into-emptyState | |
| /Users/jmatthews/scratch/2026/01_19_patternfly_5_to_6/todds_app/patternfly-migration-workshop/src/components/tier2-moderate/IconButtons.tsx | |
| 30:9 error Icons must now be passed to the `icon` prop of Button instead of as children. If you are passing anything other than an icon as children, ignore this rule when running fixes @patternfly/pf-codemods/button-moveIcons-icon-prop | |
| 37:9 error Icons must now be passed to the `icon` prop of Button instead of as children. If you are passing anything other than an icon as children, ignore this rule when running fixes @patternfly/pf-codemods/button-moveIcons-icon-prop | |
| 44:9 error Icons must now be passed to the `icon` prop of Button instead of as children. If you are passing anything other than an icon as children, ignore this rule when running fixes @patternfly/pf-codemods/button-moveIcons-icon-prop | |
| /Users/jmatthews/scratch/2026/01_19_patternfly_5_to_6/todds_app/patternfly-migration-workshop/src/components/tier3-edge-cases/CompatibilityLayer.tsx | |
| 3:1 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| 20:10 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| 20:40 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| /Users/jmatthews/scratch/2026/01_19_patternfly_5_to_6/todds_app/patternfly-migration-workshop/src/components/tier3-edge-cases/CustomWrapper.tsx | |
| 2:1 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| 7:13 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| 18:13 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| 21:5 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| 26:5 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| 35:28 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| /Users/jmatthews/scratch/2026/01_19_patternfly_5_to_6/todds_app/patternfly-migration-workshop/src/components/tier3-edge-cases/DynamicComponent.tsx | |
| 3:1 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| 37:9 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| 39:9 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| 42:12 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| 42:48 error We have replaced Text, TextContent, TextList and TextListItem with one Content component. Running this fix will change all of those components names to Content and add a `component` prop where necessary @patternfly/pf-codemods/text-replace-with-content | |
| /Users/jmatthews/scratch/2026/01_19_patternfly_5_to_6/todds_app/patternfly-migration-workshop/src/pages/ProjectsPage.tsx | |
| 20:7 warning Classes from the `variant` prop will now only be applied to PageSection when the `type` prop has a value of "default" @patternfly/pf-codemods/pageSection-warn-variantClasses-applied | |
| 20:7 error The `variant` prop for PageSection now only accepts a value of "default" or "secondary". Running the fix for this rule will remove the prop so it uses the default value of "default" @patternfly/pf-codemods/pageSection-update-variant-values | |
| 20:7 warning The isWidthLimited prop on PageSection will no longer determine whether the children are wrapped in a PageBody. Instead the new hasBodyWrapper prop must be used. By default this new prop is set to true. Running the fix for this rule will apply hasBodyWrapper with the same value as the isWidthLimited prop or false if isWidthLimited is not passed @patternfly/pf-codemods/pageBreadcrumbAndSection-warn-updated-wrapperLogic | |
| 33:7 warning The isWidthLimited prop on PageSection will no longer determine whether the children are wrapped in a PageBody. Instead the new hasBodyWrapper prop must be used. By default this new prop is set to true. Running the fix for this rule will apply hasBodyWrapper with the same value as the isWidthLimited prop or false if isWidthLimited is not passed @patternfly/pf-codemods/pageBreadcrumbAndSection-warn-updated-wrapperLogic | |
| /Users/jmatthews/scratch/2026/01_19_patternfly_5_to_6/todds_app/patternfly-migration-workshop/src/pages/StoragePage.tsx | |
| 23:7 warning Classes from the `variant` prop will now only be applied to PageSection when the `type` prop has a value of "default" @patternfly/pf-codemods/pageSection-warn-variantClasses-applied | |
| 23:7 error The `variant` prop for PageSection now only accepts a value of "default" or "secondary". Running the fix for this rule will remove the prop so it uses the default value of "default" @patternfly/pf-codemods/pageSection-update-variant-values | |
| 23:7 warning The isWidthLimited prop on PageSection will no longer determine whether the children are wrapped in a PageBody. Instead the new hasBodyWrapper prop must be used. By default this new prop is set to true. Running the fix for this rule will apply hasBodyWrapper with the same value as the isWidthLimited prop or false if isWidthLimited is not passed @patternfly/pf-codemods/pageBreadcrumbAndSection-warn-updated-wrapperLogic | |
| 36:7 warning The isWidthLimited prop on PageSection will no longer determine whether the children are wrapped in a PageBody. Instead the new hasBodyWrapper prop must be used. By default this new prop is set to true. Running the fix for this rule will apply hasBodyWrapper with the same value as the isWidthLimited prop or false if isWidthLimited is not passed @patternfly/pf-codemods/pageBreadcrumbAndSection-warn-updated-wrapperLogic | |
| /Users/jmatthews/scratch/2026/01_19_patternfly_5_to_6/todds_app/patternfly-migration-workshop/src/pages/WorkloadsPage.tsx | |
| 2:1 warning The markup for the Tabs scroll buttons has been updated. They are now rendered with a div wrapper, use our Button component, and no longer have adjusted styling when the `isSubtab` prop is true @patternfly/pf-codemods/tabs-update-markup | |
| 34:7 warning Classes from the `variant` prop will now only be applied to PageSection when the `type` prop has a value of "default" @patternfly/pf-codemods/pageSection-warn-variantClasses-applied | |
| 34:7 error The `variant` prop for PageSection now only accepts a value of "default" or "secondary". Running the fix for this rule will remove the prop so it uses the default value of "default" @patternfly/pf-codemods/pageSection-update-variant-values | |
| 34:7 warning The isWidthLimited prop on PageSection will no longer determine whether the children are wrapped in a PageBody. Instead the new hasBodyWrapper prop must be used. By default this new prop is set to true. Running the fix for this rule will apply hasBodyWrapper with the same value as the isWidthLimited prop or false if isWidthLimited is not passed @patternfly/pf-codemods/pageBreadcrumbAndSection-warn-updated-wrapperLogic | |
| 50:7 warning The isWidthLimited prop on PageSection will no longer determine whether the children are wrapped in a PageBody. Instead the new hasBodyWrapper prop must be used. By default this new prop is set to true. Running the fix for this rule will apply hasBodyWrapper with the same value as the isWidthLimited prop or false if isWidthLimited is not passed @patternfly/pf-codemods/pageBreadcrumbAndSection-warn-updated-wrapperLogic | |
| ✖ 56 problems (44 errors, 12 warnings) | |
| 40 errors and 6 warnings potentially fixable with the `--fix` option. | |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment