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

Combined changes from most fork #28

Open
wants to merge 107 commits into
base: develop
Choose a base branch
from
Open

Conversation

athimer
Copy link

@athimer athimer commented Jun 10, 2018

I was trying to combine changes from most fork to get one updated version


This change is Reviewable

mjunginger and others added 30 commits April 15, 2014 11:35
I a new configuration element called 'excludesRegex'. You can use it just as 'excludes' however it is interpreted as an array of regular expressions.
This allows e. g. to exclude all apache projects. Example:
<excludesRegex>
	<param>org.slf4j:.*</param><!-- Not declared, but is apache-2.0 -->
</excludesRegex>

Furthermore I converted the excludes and blacklist into a HashSet to avoid redundant looping over the arrays.
Allow exclusions by regular expressions
…s from check since they will not be shipped / used in production code.
Added Whitelist Feature and Exclude No License Feature
Proposed fix for log problems as warnings mrice#18
Jenkins and others added 29 commits January 9, 2017 13:33
 - remove changes only useful for company that did extension of project
 - copy some test and make pass
# Conflicts:
#	pom.xml
#	src/main/java/org/complykit/licensecheck/mojo/OpenSourceLicenseCheckMojo.java
#	src/main/resources/licenses.txt
#	src/test/java/org.complykit.licensecheck.mojo/OpenSourceLicenseCheckMojoTest.java
@mrice
Copy link
Owner

mrice commented Jun 13, 2018

Hey @athimer - thanks for this! This is a big PR. Can you give me a little more context on what you're seeking to do with it?

(Also, candidly, I'll have to remember how to do a nexus deployment for a new version. It's been a few years, to say the least............!!)

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

Successfully merging this pull request may close these issues.

6 participants