Skip to content

Conversation

@remarkablemark
Copy link
Owner

@remarkablemark remarkablemark commented Nov 18, 2016

Since the fix in #30 uses a try-catch to require the DOM property configs either from react/lib/ or react-dom/lib/, this causes webpack to throw an error (albeit harmless):

ERROR in ./lib/property-config.js
Module not found: Error: Cannot resolve module 'react/lib/HTMLDOMPropertyConfig'

After some discussion with @struesdell in #29, semver will be used to account for react latest and <15.4.

Tasks:

  • Consolidate the require logic in lib/property-config.js
  • Revert to use react@15.3 for v0.2.0 of this package
    • Update devDependencies to fix Travis tests
  • Tidy package.json and update npm scripts
  • Update README installation instructions

`./lib/html-to-dom-server.js` has been replaced with
`html-dom-parser` package.
Remove the try-catch and require the DOM property configs from
`react/lib/`.

> ERROR in ./lib/property-config.js
> Module not found: Error: Cannot resolve module
> 'react/lib/HTMLDOMPropertyConfig'
Rename `build` to `build-unmin` and update `prepublish` to run
`clean` before the build scripts.
@remarkablemark remarkablemark self-assigned this Nov 18, 2016
@coveralls
Copy link

Coverage Status

Coverage increased (+1.4%) to 99.231% when pulling dfb86c2 on react@15.3 into b809226 on master.

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.

3 participants