Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

make it possible to define lines in licenseFileOverrides #121

Open
mojoaxel opened this issue Mar 18, 2022 · 2 comments
Open

make it possible to define lines in licenseFileOverrides #121

mojoaxel opened this issue Mar 18, 2022 · 2 comments

Comments

@mojoaxel
Copy link

We needed to define some license-file mappings in the [licenseFileOverrides option] because some (old version) of some libs had their license statement not inside a seperate file, but in the README.

So we needed to set something like this:

licenseFileOverrides: {
	'cookie-signature': 'Readme.md',
	isarray: 'README.md',
}

This results in the (whole) README to be used as license text.

It would be better is we could define some lines inside a file and only use these lines as the license text.
We would try to provide an implementation for that over the next weeks.

Please vote on the implementation details:

@mojoaxel
Copy link
Author

We propose this way to implement this:

licenseFileOverrides: {
	'cookie-signature': 'Readme.md/#L21-L42',
	isarray: 'README.md#L40-L60',
}

@mojoaxel
Copy link
Author

Alternatively we could make it more verbose:

licenseFileOverrides: {
	'cookie-signature': {
		file: 'Readme.md',
		lines: {
			from: 21,
			to: 42 // optional
		}
	}
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant