Skip to content
This repository has been archived by the owner on Mar 17, 2021. It is now read-only.

fix(schema): allow additionalProperties #207

Merged
merged 2 commits into from
Sep 30, 2017
Merged

fix(schema): allow additionalProperties #207

merged 2 commits into from
Sep 30, 2017

Conversation

michael-ciniawsky
Copy link
Member

Noteable Changes


Allow addtionalProperties for url-loader fallback compat

Issues


@codecov
Copy link

codecov bot commented Sep 30, 2017

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #207   +/-   ##
=======================================
  Coverage   96.96%   96.96%           
=======================================
  Files           2        2           
  Lines          33       33           
  Branches       16       16           
=======================================
  Hits           32       32           
  Misses          1        1

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 abe91a8...794357f. Read the comment docs.

@joshwiens
Copy link
Member

@michael-ciniawsky - Shipping this as we have a broken minor version on npm

appveyor.yml Outdated
job_part: test
build: 'off'
matrix:
fast_finish: true
install:
- ps: Install-Product node $env:nodejs_version x64
- 'ps: Install-Product node $env:nodejs_version x64'
Copy link
Member

Choose a reason for hiding this comment

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

Pull the single quotes off all the powershell commands

.eslintrc Outdated
}
}
"extends": "webpack"
}
Copy link
Member

Choose a reason for hiding this comment

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

Roll this back to "no-undefined": 0

@michael-ciniawsky
Copy link
Member Author

kk mom :)

@joshwiens
Copy link
Member

And don't forget to brush your teeth before bed .....

@michael-ciniawsky
Copy link
Member Author

mom === just a moment

Is this uncommon to use ? 🙃 😕

@joshwiens
Copy link
Member

Evidently I'm just getting old ...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants