small updates to node-schema-org #2

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
@femto113
Contributor

femto113 commented Feb 24, 2012

Hey there, this utility turned out to be quite useful for something I'm working on, but it needed a couple tweaks to work for me so I thought I'd share them back with you. First commit has the functional changes: a couple command line enhancements to the read-schema-org script to control concurrency (75 ground my laptop to a halt) and limit the parsing to specific schemas (useful for testing); also made some small enhancements to the parsing in read.js, primarily adding the version info from the source HTML. The second commit is just the latest schema files run with this version of the parser. --Ken

femto113 added some commits Feb 24, 2012

- added command line control over concurrency and a saner default
- added support for specifying list of schemas on command line
- added parsing of schema version
- fixed whitespace issue in property type parsing
@indexzero

This comment has been minimized.

Show comment Hide comment
@indexzero

indexzero Jan 18, 2013

Owner

Cherry-picked. Thanks! Sorry it took so long to merge. I don't use this library often.

Owner

indexzero commented Jan 18, 2013

Cherry-picked. Thanks! Sorry it took so long to merge. I don't use this library often.

@indexzero indexzero closed this Jan 18, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment