do not migrate language to matchLanguage

secustor · Jul 1, 2023 · 2e11610 · 2e11610
1 parent 6abe789
commit 2e11610
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/lib/config/migration.spec.ts b/lib/config/migration.spec.ts
@@ -539,7 +539,7 @@ describe('config/migration', () => {
         packageRules: [
           {
             paths: ['package.json'],
-            languages: ['python'],
+            matchLanguages: ['python'],
             baseBranchList: ['master'],
             managers: ['dockerfile'],
             datasources: ['orb'],

diff --git a/lib/config/migrations/custom/package-rules-migration.ts b/lib/config/migrations/custom/package-rules-migration.ts
@@ -6,7 +6,6 @@ export const renameMap = {
   matchFiles: 'matchFileNames',
   matchPaths: 'matchFileNames',
   paths: 'matchFileNames',
-  languages: 'matchLanguages',
   baseBranchList: 'matchBaseBranches',
   managers: 'matchManagers',
   datasources: 'matchDatasources',
@@ -29,6 +28,7 @@ function renameKeys(packageRule: PackageRule): PackageRule {
 function removeMatchLanguage(packageRule: PackageRule): PackageRule[] {
   const newPackageRules: PackageRule[] = [];
   const matchLanguages = packageRule.matchLanguages;
+  delete packageRule.matchLanguages;
   // no migration needed
   if (
     is.nullOrUndefined(matchLanguages) ||
@@ -40,7 +40,6 @@ function removeMatchLanguage(packageRule: PackageRule): PackageRule[] {
 
   // deep copy
   const newRule: PackageRule = structuredClone(packageRule);
-  delete newRule.matchLanguages;
 
   // are there any 1:1 migrateable languages
   newRule.matchCategories = matchLanguages;