Skip to content
Snippets Groups Projects
Unverified Commit a9a81275 authored by RahulGautamSingh's avatar RahulGautamSingh Committed by GitHub
Browse files

refactor: runtime-config-narrowing (#15897)

* refactor/runtime-config-narrowing

* remove test

* apply fixes

* reorder tests
parent 4b6fc851
No related branches found
No related tags found
No related merge requests found
...@@ -228,6 +228,8 @@ export interface RenovateConfig ...@@ -228,6 +228,8 @@ export interface RenovateConfig
fetchReleaseNotes?: boolean; fetchReleaseNotes?: boolean;
secrets?: Record<string, string>; secrets?: Record<string, string>;
aliases?: Record<string, any>;
skipInstalls?: boolean;
} }
export interface AllConfig extends RenovateConfig, GlobalOnlyConfig {} export interface AllConfig extends RenovateConfig, GlobalOnlyConfig {}
......
...@@ -6,13 +6,26 @@ import { ...@@ -6,13 +6,26 @@ import {
extractPackageFile, extractPackageFile,
get, get,
} from '../../../modules/manager'; } from '../../../modules/manager';
import type { PackageFile } from '../../../modules/manager/types'; import type {
ExtractConfig,
PackageFile,
} from '../../../modules/manager/types';
import { readLocalFile } from '../../../util/fs'; import { readLocalFile } from '../../../util/fs';
function getExtractConfig(config: WorkerExtractConfig): ExtractConfig {
return {
npmrc: config.npmrc,
aliases: config.aliases,
skipInstalls: config.skipInstalls,
npmrcMerge: config.npmrcMerge,
};
}
export async function getManagerPackageFiles( export async function getManagerPackageFiles(
config: WorkerExtractConfig config: WorkerExtractConfig
): Promise<PackageFile[]> { ): Promise<PackageFile[]> {
const { enabled, manager, fileList } = config; const { enabled, manager, fileList } = config;
const extractConfig = getExtractConfig(config);
logger.trace(`getPackageFiles(${manager})`); logger.trace(`getPackageFiles(${manager})`);
if (!enabled) { if (!enabled) {
logger.debug(`${manager} is disabled`); logger.debug(`${manager} is disabled`);
...@@ -32,7 +45,7 @@ export async function getManagerPackageFiles( ...@@ -32,7 +45,7 @@ export async function getManagerPackageFiles(
if (get(manager, 'extractAllPackageFiles')) { if (get(manager, 'extractAllPackageFiles')) {
const allPackageFiles = await extractAllPackageFiles( const allPackageFiles = await extractAllPackageFiles(
manager, manager,
config, extractConfig,
fileList fileList
); );
if (allPackageFiles) { if (allPackageFiles) {
...@@ -54,7 +67,7 @@ export async function getManagerPackageFiles( ...@@ -54,7 +67,7 @@ export async function getManagerPackageFiles(
manager, manager,
content, content,
packageFile, packageFile,
config extractConfig
); );
if (res) { if (res) {
for (let index = 0; index < res.deps.length; index += 1) { for (let index = 0; index < res.deps.length; index += 1) {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment