GitHub PullRequestReviewCommentEvent at 2014-03-26 14:23:12
Repository: spree
Actor: bbuchalter
Payload: {"comment":{"url":"https://api.github.com/repos/spree/spree/pulls/comments/10977998","id":10977998,"diff_hunk":"@@ -203,6 +203,9 @@ generate test applications and run specs for all the facets:\n bash build.sh\n ```\n \n+In order for JavaScript dependent request specs to run, you must use\n+[Firefox version 25](https://ftp.mozilla.org/pub/mozilla.org/firefox/releases/25.0/) or earlier.","path":"README.md","position":5,"original_position":5,"commit_id":"aa8f1e55c35a8c9612545b648795d70aa2b977fe","original_commit_id":"aa8f1e55c35a8c9612545b648795d70aa2b977fe","user":{"login":"bbuchalter","id":540762,"avatar_url":"https://avatars.githubusercontent.com/u/540762?","gravatar_id":"46cf138fbe8f435d8ace342ad47ab3ae","url":"https://api.github.com/users/bbuchalter","html_url":"https://github.com/bbuchalter","followers_url":"https://api.github.com/users/bbuchalter/followers","following_url":"https://api.github.com/users/bbuchalter/following{/other_user}","gists_url":"https://api.github.com/users/bbuchalter/gists{/gist_id}","starred_url":"https://api.github.com/users/bbuchalter/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bbuchalter/subscriptions","organizations_url":"https://api.github.com/users/bbuchalter/orgs","repos_url":"https://api.github.com/users/bbuchalter/repos","events_url":"https://api.github.com/users/bbuchalter/events{/privacy}","received_events_url":"https://api.github.com/users/bbuchalter/received_events","type":"User","site_admin":false},"body":"Looks like this is a [known issue](https://code.google.com/p/selenium/issues/detail?id=6517) with selenium-webdriver on which capybara depends. So I guess it's probably not appropriate to include this workaround? Or maybe highlight that it is currently a work around and should go away sometime in the future? Perhaps it's better than nothing until the dust settles with these dependencies. Your call...I'm happy to do whatever works.","created_at":"2014-03-26T14:23:12Z","updated_at":"2014-03-26T14:23:12Z","html_url":"https://github.com/spree/spree/pull/3325#discussion_r10977998","pull_request_url":"https://api.github.com/repos/spree/spree/pulls/3325","_links":{"self":{"href":"https://api.github.com/repos/spree/spree/pulls/comments/10977998"},"html":{"href":"https://github.com/spree/spree/pull/3325#discussion_r10977998"},"pull_request":{"href":"https://api.github.com/repos/spree/spree/pulls/3325"}}}}