{"action":"created","issue":{"url":"https://api.github.com/repos/npm/marky-markdown/issues/114","labels_url":"https://api.github.com/repos/npm/marky-markdown/issues/114/labels{/name}","comments_url":"https://api.github.com/repos/npm/marky-markdown/issues/114/comments","events_url":"https://api.github.com/repos/npm/marky-markdown/issues/114/events","html_url":"https://github.com/npm/marky-markdown/pull/114","id":126930739,"number":114,"title":"Overhaul devDependencies' readme parsing test strategy","user":{"login":"revin","id":94850,"avatar_url":"https://avatars.githubusercontent.com/u/94850?v=3","gravatar_id":"","url":"https://api.github.com/users/revin","html_url":"https://github.com/revin","followers_url":"https://api.github.com/users/revin/followers","following_url":"https://api.github.com/users/revin/following{/other_user}","gists_url":"https://api.github.com/users/revin/gists{/gist_id}","starred_url":"https://api.github.com/users/revin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/revin/subscriptions","organizations_url":"https://api.github.com/users/revin/orgs","repos_url":"https://api.github.com/users/revin/repos","events_url":"https://api.github.com/users/revin/events{/privacy}","received_events_url":"https://api.github.com/users/revin/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":10,"created_at":"2016-01-15T18:35:27Z","updated_at":"2016-01-19T14:57:58Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/npm/marky-markdown/pulls/114","html_url":"https://github.com/npm/marky-markdown/pull/114","diff_url":"https://github.com/npm/marky-markdown/pull/114.diff","patch_url":"https://github.com/npm/marky-markdown/pull/114.patch"},"body":"A bunch of our `devDependencies` are only included so we can use their README data as test fixtures. This commit adds their READMEs as static files we load separately during `npm test` and gets rid of their entries in `package.json`. As far as I can tell, we only need `mocha`, `standard`, and `glob` for actual development.\r\n\r\nThe overall testing strategy is similar to what we already have, except instead of loading all of the markdown strings in `fixtures.packageNames` we split them into `fixtures.dependencies` (which may be a poor name, since it *does* also include the `devDependencies` as well) and `fixtures.examples`. This allows us to differentiate them during test runs because we don't have to try to `require()` the static fixtures' packages; we can just mock them as `{name: name}`.\r\n\r\nUnless there's more to this than I could glean just by reading the issue itself, this fixes #91."},"comment":{"url":"https://api.github.com/repos/npm/marky-markdown/issues/comments/172877873","html_url":"https://github.com/npm/marky-markdown/pull/114#issuecomment-172877873","issue_url":"https://api.github.com/repos/npm/marky-markdown/issues/114","id":172877873,"user":{"login":"ashleygwilliams","id":1163554,"avatar_url":"https://avatars.githubusercontent.com/u/1163554?v=3","gravatar_id":"","url":"https://api.github.com/users/ashleygwilliams","html_url":"https://github.com/ashleygwilliams","followers_url":"https://api.github.com/users/ashleygwilliams/followers","following_url":"https://api.github.com/users/ashleygwilliams/following{/other_user}","gists_url":"https://api.github.com/users/ashleygwilliams/gists{/gist_id}","starred_url":"https://api.github.com/users/ashleygwilliams/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ashleygwilliams/subscriptions","organizations_url":"https://api.github.com/users/ashleygwilliams/orgs","repos_url":"https://api.github.com/users/ashleygwilliams/repos","events_url":"https://api.github.com/users/ashleygwilliams/events{/privacy}","received_events_url":"https://api.github.com/users/ashleygwilliams/received_events","type":"User","site_admin":false},"created_at":"2016-01-19T14:57:58Z","updated_at":"2016-01-19T14:57:58Z","body":"this looks great and i'm super excited about it. thanks so much @revin!"}}