Skip to content
This repository has been archived by the owner on Apr 8, 2019. It is now read-only.

fix: properly pass require to config-loader #153

Merged
merged 1 commit into from
May 28, 2018
Merged

Conversation

shellscape
Copy link
Contributor

This PR contains a:

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

Motivation / Use-Case

Fixes #140. No mechanisms in place yet for tests around this change.

Breaking Changes

None

Additional Info

@codecov
Copy link

codecov bot commented May 28, 2018

Codecov Report

Merging #153 into master will increase coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #153      +/-   ##
==========================================
+ Coverage   96.64%   96.66%   +0.02%     
==========================================
  Files           7        7              
  Lines         268      270       +2     
==========================================
+ Hits          259      261       +2     
  Misses          9        9
Impacted Files Coverage Δ
lib/config.js 94.23% <100%> (+0.23%) ⬆️

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 f7842c4...1b1a031. Read the comment docs.

@shellscape shellscape merged commit acef1fb into master May 28, 2018
@shellscape shellscape deleted the fix/require branch May 28, 2018 15:32
@jdalton

This comment has been minimized.

@webpack-contrib webpack-contrib locked as off-topic and limited conversation to collaborators Jun 6, 2018
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.

--require and -r options are broken
3 participants