Skip to content

fix(@angular/cli): fix empty 3rdpartylicenses.txt - #7064

Merged
filipesilva merged 1 commit into
angular:masterfrom
xz64:fix_empty_3rdpartylicenses
Aug 4, 2017
Merged

fix(@angular/cli): fix empty 3rdpartylicenses.txt#7064
filipesilva merged 1 commit into
angular:masterfrom
xz64:fix_empty_3rdpartylicenses

Conversation

@xz64

@xz64 xz64 commented Jul 20, 2017

Copy link
Copy Markdown
Contributor

Upgrades license-webpack-plugin from 0.4.x to 0.5.x.

Closes #6921

@filipesilva
filipesilva self-requested a review July 31, 2017 13:38
@filipesilva filipesilva self-assigned this Jul 31, 2017
filipesilva
filipesilva previously approved these changes Jul 31, 2017

@filipesilva filipesilva left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Heya @xz64, thanks for fixing this issue and submitting the update here. Can you rebase please? It took us a while to review this Pr and meanwhile other dependencies changed.

@xz64

xz64 commented Aug 1, 2017

Copy link
Copy Markdown
Contributor Author

Hi @filipesilva, I have rebased the branch.

@filipesilva

Copy link
Copy Markdown
Contributor

@xz64 I'm so sorry, yesterday we made the move from yarn to npm5 (#7060) and all PRs that changed dependencies became invalidated. Can you rebase again, and not commit the yarn lock please?

filipesilva
filipesilva previously approved these changes Aug 2, 2017
Upgrades license-webpack-plugin from 0.4.x to 0.5.x.

Closes #6921
@xz64

xz64 commented Aug 4, 2017

Copy link
Copy Markdown
Contributor Author

It is rebased now, without a yarn.lock file.

@filipesilva
filipesilva merged commit 754c1e7 into angular:master Aug 4, 2017
@filipesilva

Copy link
Copy Markdown
Contributor

It's in now, thanks again for your work @xz64 👍

@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Sep 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3rdpartylicenses.txt is empty

3 participants