fix: include rule in betaRules

patternfly · Aug 28, 2024 · 4c9da6c · 4c9da6c
1 parent 359a228
commit 4c9da6c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/packages/eslint-plugin-pf-codemods/src/ruleCustomization.ts b/packages/eslint-plugin-pf-codemods/src/ruleCustomization.ts
@@ -2,7 +2,10 @@
  * add the rule name to the below array. Do not add rules here if the React PR is
  * not yet merged; instead add a "do not merge" label to the codemod PR.
  */
-export const betaRuleNames: string[] = ["kebabToggle-replace-with-menuToggle"];
+export const betaRuleNames: string[] = [
+  "data-codemods-cleanup",
+  "kebabToggle-replace-with-menuToggle",
+];
 
 // if you want a rule to have a severity that defaults to warning rather than error, add the rule name to the below array
 export const warningRules = [

diff --git a/packages/pf-codemods/index.js b/packages/pf-codemods/index.js
@@ -81,9 +81,6 @@ function getRulesToRemove(options) {
 async function runCodemods(path, otherPaths, options) {
   const prefix = "@patternfly/pf-codemods/";
 
-  // data-codemods-cleanup rule won't be included in the recommended rule set
-  delete configs.recommended.rules[prefix + "data-codemods-cleanup"];
-
   if (options.only) {
     // Set rules to error like eslint likes
     configs.recommended.rules = options.only.split(",").reduce((acc, rule) => {