Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Fix warning in package.json: change bugs[web] to bugs[url] #39

Closed
wants to merge 1 commit into from

2 participants

@jmalonzo

No description provided.

@fb55

The fourth fix for that issue… This project doesn't seem to be continued, so it won't get merged. (But you may use my fork, that is still actively developed and has that bug fixed by default.)

@jmalonzo

The author just updated the license last month. It would be nice if your fork is the one that gets picked up by npm. Have you tried talking/emailing the author and perhaps just take ownership/maintainership of the project?

@fb55

I've added my fork as "htmlparser2" to npm. Project relying on htmlparser just need to change their requires to htmlparser2 and they are ready to go.

Taking ownership of this project may not be the best idea. I removed browser support as I don't need it (they already have an HTML parser, don't they?) and introduced some breaking changes (eg. a new internal structure for handlers, no more verbose output), so any project using those features may be broken.

@jmalonzo

Thanks. Fair enough. I'll keep that in mind if I see a project that uses htmlparser.

Cheers.

@jmalonzo jmalonzo closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Dec 4, 2011
  1. @jmalonzo
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  package.json
View
2  package.json
@@ -10,7 +10,7 @@
}
, "bugs": {
"mail": "chris@winberry.net"
- , "web": "http://github.com/tautologistics/node-htmlparser/issues"
+ , "url": "http://github.com/tautologistics/node-htmlparser/issues"
}
, "os": [ "linux", "darwin", "freebsd", "win32" ]
, "directories": { "lib": "./lib/" }
Something went wrong with that request. Please try again.