Skip to content

Print files with unapproved licenses in travis#6947

Merged
fjy merged 1 commit intoapache:masterfrom
jihoonson:print-rat-output
Jan 29, 2019
Merged

Print files with unapproved licenses in travis#6947
fjy merged 1 commit intoapache:masterfrom
jihoonson:print-rat-output

Conversation

@jihoonson
Copy link
Copy Markdown
Contributor

This PR shows the list of files with unapproved licenses after running Apache rat in Travis, so that we don't have to run again manually to see them.

The below is an example output. I tested with #6923.

...
[INFO] 118 resources included (use -debug for more details)
[WARNING] Files with unapproved licenses:
  /Users/jihoonson/Codes/druid/web-console/legacy/druid.css
  /Users/jihoonson/Codes/druid/web-console/legacy/druid.js
  /Users/jihoonson/Codes/druid/web-console/lib/react-table.styl
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary for Druid 0.13.0-incubating-SNAPSHOT:
[INFO] 
[INFO] Druid .............................................. SUCCESS [  6.880 s]
[INFO] druid-core ......................................... SUCCESS [ 24.825 s]
[INFO] druid-hll .......................................... SUCCESS [  5.012 s]
[INFO] extendedset ........................................ SUCCESS [  4.893 s]
[INFO] druid-processing ................................... SUCCESS [ 25.595 s]
[INFO] druid-aws-common ................................... SUCCESS [  4.531 s]
[INFO] druid-console ...................................... FAILURE [ 32.699 s]
...

Copy link
Copy Markdown
Contributor

@gianm gianm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM after CI

@fjy fjy merged commit c23c5ef into apache:master Jan 29, 2019
justinborromeo pushed a commit to justinborromeo/incubator-druid that referenced this pull request Feb 2, 2019
@jon-wei jon-wei added this to the 0.14.0 milestone Feb 20, 2019
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.

4 participants