Also find lint-staged config in package.yaml manifest (support pnpm)

[danielbayley]

Description

Since pnpm supports package.yaml (see: pnpm/pnpm#1799), it would be useful if a lint-staged property would also be respected from this alternative manifest format. json5 is also a possibility.

Since this project already supports reading config from YAML files, this should be an easy win…

Environment

• OS: macOS Ventura
• Node.js: v21.1.0
• lint-staged: v15.0.2

[iiroj]

Can you try if it starts working by sinply adding the file name to this list?

lint-staged/lib/loadConfig.js

Lines 14 to 30 in 21aeeb5

	/**
	* The list of files `lint-staged` will read configuration
	* from, in the declared order.
	*/
	export const searchPlaces = [
	PACKAGE_JSON,
	'.lintstagedrc',
	'.lintstagedrc.json',
	'.lintstagedrc.yaml',
	'.lintstagedrc.yml',
	'.lintstagedrc.mjs',
	'.lintstagedrc.js',
	'.lintstagedrc.cjs',
	'lint-staged.config.mjs',
	'lint-staged.config.js',
	'lint-staged.config.cjs',
	]

[danielbayley]

Can you try if it starts working by simply adding the file name to this list?

@iiroj It fails with a Validation Error—Should be a string, a function, or an array of strings and functions.—thrown from validateConfig.js… The exact same manifest in package.json format however does not.

[iiroj]

Thanks, it probably assumes the file is directly the config, instead of there being the lint-staged key inside it.

[danielbayley]

Thanks, it probably assumes the file is directly the config, instead of there being the lint-staged key inside it.

@iiroj Yeah I figured. See attempted implementation in #1344…