Skip to content
This repository was archived by the owner on Dec 22, 2020. It is now read-only.

misc(readme) fix mixed content on the readme file #38

Merged
merged 2 commits into from
Apr 24, 2019

Conversation

EugeneHlushko
Copy link
Contributor

This PR contains a:

  • bugfix
  • new feature
  • code refactor
  • test update
  • typo fix
  • metadata update

Motivation / Use-Case

We are getting an alert on webpack.js.org builds, we fetch webpack contrib plugins and loaders
README.md files automatically, and this one seems to have mixed content causing that issue.

Breaking Changes

Additional Info

@alexander-akait
Copy link
Member

Need update lock file to avoid security problem

@EugeneHlushko
Copy link
Contributor Author

Do you mean i go and yarn install and commit if any changes or you updated it on master and i need a rebase? @evilebottnawi

@alexander-akait
Copy link
Member

Hi, rm -r package-lock.json && npm i and send and send a package-lock.json in this PR

@codecov
Copy link

codecov bot commented Apr 24, 2019

Codecov Report

Merging #38 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #38   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           2      2           
  Lines          11     11           
  Branches        4      4           
=====================================
  Hits           11     11

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9095862...98eb07f. Read the comment docs.

@EugeneHlushko
Copy link
Contributor Author

@evilebottnawi done

@alexander-akait alexander-akait merged commit b4155dd into webpack-contrib:master Apr 24, 2019
@alexander-akait
Copy link
Member

Thanks!

@EugeneHlushko EugeneHlushko deleted the fix-https branch April 24, 2019 11:16
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.

2 participants