{"action":"created","issue":{"url":"https://api.github.com/repos/edx/xblock-sdk/issues/91","labels_url":"https://api.github.com/repos/edx/xblock-sdk/issues/91/labels{/name}","comments_url":"https://api.github.com/repos/edx/xblock-sdk/issues/91/comments","events_url":"https://api.github.com/repos/edx/xblock-sdk/issues/91/events","html_url":"https://github.com/edx/xblock-sdk/pull/91","id":118067053,"number":91,"title":"removed the Acid Block from the test requirements","user":{"login":"Lyla-Fischer","id":549661,"avatar_url":"https://avatars.githubusercontent.com/u/549661?v=3","gravatar_id":"","url":"https://api.github.com/users/Lyla-Fischer","html_url":"https://github.com/Lyla-Fischer","followers_url":"https://api.github.com/users/Lyla-Fischer/followers","following_url":"https://api.github.com/users/Lyla-Fischer/following{/other_user}","gists_url":"https://api.github.com/users/Lyla-Fischer/gists{/gist_id}","starred_url":"https://api.github.com/users/Lyla-Fischer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Lyla-Fischer/subscriptions","organizations_url":"https://api.github.com/users/Lyla-Fischer/orgs","repos_url":"https://api.github.com/users/Lyla-Fischer/repos","events_url":"https://api.github.com/users/Lyla-Fischer/events{/privacy}","received_events_url":"https://api.github.com/users/Lyla-Fischer/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/edx/xblock-sdk/labels/awaiting%20prioritization","name":"awaiting prioritization","color":"fad8c7"},{"url":"https://api.github.com/repos/edx/xblock-sdk/labels/open-source-contribution","name":"open-source-contribution","color":"bfe5bf"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-11-20T15:45:07Z","updated_at":"2015-11-25T20:36:56Z","closed_at":"2015-11-25T20:36:56Z","pull_request":{"url":"https://api.github.com/repos/edx/xblock-sdk/pulls/91","html_url":"https://github.com/edx/xblock-sdk/pull/91","diff_url":"https://github.com/edx/xblock-sdk/pull/91.diff","patch_url":"https://github.com/edx/xblock-sdk/pull/91.patch"},"body":"\r\n\r\nRemoves the Acid Block from the test requirements in the XBlock SDK. The justification is as follows:\r\n\r\n(1) The status quo requires developers to break from default testing procedure in order to use the Acid Block effectively as a testing tool, so asking developers to install the Acid Block separately from a default installation is not an unreasonable requirement.\r\n\r\nAs a reminder, the Acid Block does not integrate with the testing framework that developers are using in the XBlock SDK because it does not produce a little green dot on the command line when running \"make test\". The acid block is currently installed by default when the developer uses \"make\" or \"make install\", because those commands install all of the test dependencies, including the acid block.\r\n\r\n(2) Furthermore, the Aside functionality within the block causes it to display prominently during non-testing, which means that it's installation as default behaviour is causing a lot of confusion and annoyance.\r\n\r\nSee full discussion from edx-code here: https://groups.google.com/forum/#!topic/edx-code/r-xEVGNjJhM\r\n"},"comment":{"url":"https://api.github.com/repos/edx/xblock-sdk/issues/comments/159725535","html_url":"https://github.com/edx/xblock-sdk/pull/91#issuecomment-159725535","issue_url":"https://api.github.com/repos/edx/xblock-sdk/issues/91","id":159725535,"user":{"login":"nedbat","id":23789,"avatar_url":"https://avatars.githubusercontent.com/u/23789?v=3","gravatar_id":"","url":"https://api.github.com/users/nedbat","html_url":"https://github.com/nedbat","followers_url":"https://api.github.com/users/nedbat/followers","following_url":"https://api.github.com/users/nedbat/following{/other_user}","gists_url":"https://api.github.com/users/nedbat/gists{/gist_id}","starred_url":"https://api.github.com/users/nedbat/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nedbat/subscriptions","organizations_url":"https://api.github.com/users/nedbat/orgs","repos_url":"https://api.github.com/users/nedbat/repos","events_url":"https://api.github.com/users/nedbat/events{/privacy}","received_events_url":"https://api.github.com/users/nedbat/received_events","type":"User","site_admin":false},"created_at":"2015-11-25T20:36:56Z","updated_at":"2015-11-25T20:36:56Z","body":"@Lyla-Fischer Can you give me some scenarios to try here?  The acid block is meant to help with testing whether XBlocks are supported correctly.  I would have thought we need them here for testing the SDK itself.  "}}