Skip to content

Commit

Permalink
following #106, makes options.publicPath required via the API. alphab…
Browse files Browse the repository at this point in the history
…etizes optionsSchema (#1055)

* following #106, makes options.publicPath required via the API. alphabetizes optionsSchema

* fixing options validation tests

* reorg options schema

* refactoring options validation test to prevent needless duplication
  • Loading branch information
shellscape authored Sep 27, 2017
1 parent 1c9881e commit ed2d97b
Show file tree
Hide file tree
Showing 5 changed files with 282 additions and 214 deletions.
2 changes: 1 addition & 1 deletion lib/OptionsValidationError.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

/* eslint no-param-reassign: 'off' */

const optionsSchema = require('./optionsSchema.json');
const optionsSchema = require('./schemas/options.json');

const indent = (str, prefix, firstLine) => {
if (firstLine) {
Expand Down
2 changes: 1 addition & 1 deletion lib/Server.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ const spdy = require('spdy');
const webpack = require('webpack');
const webpackDevMiddleware = require('webpack-dev-middleware');
const OptionsValidationError = require('./OptionsValidationError');
const optionsSchema = require('./optionsSchema.json');
const optionsSchema = require('./schemas/options.json');

const clientStats = { errorDetails: false };
const log = console.log; // eslint-disable-line no-console
Expand Down
Loading

0 comments on commit ed2d97b

Please sign in to comment.