Skip to content

/loaders: add index of important loaders#804

Merged
bebraw merged 2 commits intowebpack:masterfrom
simon04:loader-list
Feb 5, 2017
Merged

/loaders: add index of important loaders#804
bebraw merged 2 commits intowebpack:masterfrom
simon04:loader-list

Conversation

@simon04
Copy link
Copy Markdown
Contributor

@simon04 simon04 commented Feb 3, 2017

@bebraw
Copy link
Copy Markdown
Contributor

bebraw commented Feb 4, 2017

I wonder what to do with those license links. I guess we should resolve them to the packages somehow as now it's trying to look up against the site itself.

@simon04
Copy link
Copy Markdown
Contributor Author

simon04 commented Feb 4, 2017

The idea of those license links/checks is to make sure that the included content is under a compatible license?

@bebraw
Copy link
Copy Markdown
Contributor

bebraw commented Feb 4, 2017

The links are broken in npm too due to the way they have been written. I guess the best fix would be to fix the links at each package for this reason rather than work around here.

@simon04
Copy link
Copy Markdown
Contributor Author

simon04 commented Feb 4, 2017

Hopefully those PRs will be merged quickly 😄

@bebraw
Copy link
Copy Markdown
Contributor

bebraw commented Feb 5, 2017

@simon04 Merged the PRs.

@simon04
Copy link
Copy Markdown
Contributor Author

simon04 commented Feb 5, 2017

Thank you! Now the Travis build succeeds.

@bebraw bebraw merged commit d06d031 into webpack:master Feb 5, 2017
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.

2 participants