{"action":"created","issue":{"url":"https://api.github.com/repos/exercism/xelm/issues/76","repository_url":"https://api.github.com/repos/exercism/xelm","labels_url":"https://api.github.com/repos/exercism/xelm/issues/76/labels{/name}","comments_url":"https://api.github.com/repos/exercism/xelm/issues/76/comments","events_url":"https://api.github.com/repos/exercism/xelm/issues/76/events","html_url":"https://github.com/exercism/xelm/issues/76","id":154940964,"number":76,"title":"runtests.sh is not downloaded with executable permissions","user":{"login":"tgecho","id":49615,"avatar_url":"https://avatars.githubusercontent.com/u/49615?v=3","gravatar_id":"","url":"https://api.github.com/users/tgecho","html_url":"https://github.com/tgecho","followers_url":"https://api.github.com/users/tgecho/followers","following_url":"https://api.github.com/users/tgecho/following{/other_user}","gists_url":"https://api.github.com/users/tgecho/gists{/gist_id}","starred_url":"https://api.github.com/users/tgecho/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tgecho/subscriptions","organizations_url":"https://api.github.com/users/tgecho/orgs","repos_url":"https://api.github.com/users/tgecho/repos","events_url":"https://api.github.com/users/tgecho/events{/privacy}","received_events_url":"https://api.github.com/users/tgecho/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2016-05-16T00:21:54Z","updated_at":"2016-07-15T13:02:49Z","closed_at":null,"body":"```\r\n$ ./runtests.sh\r\n-bash: ./runtests.sh: Permission denied\r\n```\r\n\r\nI didn't even think to check/ask, but it looks like this behavior was changed a few months ago: https://github.com/exercism/cli/issues/276 :(\r\n\r\nSo... seeing as in my haste I've failed to create a clean/generic wrapper on the first try, I see a few obvious options:\r\n\r\n1. Remove the wrappers, switch the instructions back and wait for an updated `elm-test` to ship. `elm-test MyTests.elm` actually works right now, it just has a bunch of extraneous garbage at the end of the output.\r\n2. Update the instructions to `bash runtests.sh`.\r\n3. Write a node.js based wrapper (which could actually be cross platform) and instruct them to run `node runtests.js`.\r\n\r\nThoughts @parkerl @lukewestby?"},"comment":{"url":"https://api.github.com/repos/exercism/xelm/issues/comments/232945266","html_url":"https://github.com/exercism/xelm/issues/76#issuecomment-232945266","issue_url":"https://api.github.com/repos/exercism/xelm/issues/76","id":232945266,"user":{"login":"tgecho","id":49615,"avatar_url":"https://avatars.githubusercontent.com/u/49615?v=3","gravatar_id":"","url":"https://api.github.com/users/tgecho","html_url":"https://github.com/tgecho","followers_url":"https://api.github.com/users/tgecho/followers","following_url":"https://api.github.com/users/tgecho/following{/other_user}","gists_url":"https://api.github.com/users/tgecho/gists{/gist_id}","starred_url":"https://api.github.com/users/tgecho/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tgecho/subscriptions","organizations_url":"https://api.github.com/users/tgecho/orgs","repos_url":"https://api.github.com/users/tgecho/repos","events_url":"https://api.github.com/users/tgecho/events{/privacy}","received_events_url":"https://api.github.com/users/tgecho/received_events","type":"User","site_admin":false},"created_at":"2016-07-15T13:02:49Z","updated_at":"2016-07-15T13:02:49Z","body":"@Calvyn82 thanks for reminding me about this! I'm really unhappy with asking people to modify the runtests scripts every time they download a new exercise.\r\n\r\nI'm almost tempted to switch back to recommending `elm-test-runner`, with the caveat that it shows an odd error message at the end. It does look like rtfeldman is planning to update soon: https://github.com/rtfeldman/node-test-runner\r\n\r\n```sh\r\n$ elm-test SayTests.elm\r\nSuccess! Compiled 1 module.\r\nSuccessfully generated /var/folders/c8/lwxn19p55rj7mq0n7xhj0hv00000gp/T/elm_test_116615-34499-1gpd4jn.js\r\nSuccessfully compiled SayTests.elm\r\nRunning tests...\r\n  1 suites run, containing 17 tests\r\n  All tests passed\r\n\r\n\r\n: \"\"\r\n\r\n[stdin]:7603\r\n    if (typeof Elm === \"undefined\") { throw \"elm-io config error: Elm is not defined. Make sure you call elm-io with a real Elm output file\"}\r\n                                      ^\r\nelm-io config error: Elm is not defined. Make sure you call elm-io with a real Elm output file\r\n```\r\n\r\nIf we want to stay self sufficient, I think I'd prefer to write a little node powered wrapper. Thoughts?"}}
{"actor":{"display_login":"tgecho"}}