{"action":"created","issue":{"url":"https://api.github.com/repos/DynamoDS/Dynamo/issues/6062","repository_url":"https://api.github.com/repos/DynamoDS/Dynamo","labels_url":"https://api.github.com/repos/DynamoDS/Dynamo/issues/6062/labels{/name}","comments_url":"https://api.github.com/repos/DynamoDS/Dynamo/issues/6062/comments","events_url":"https://api.github.com/repos/DynamoDS/Dynamo/issues/6062/events","html_url":"https://github.com/DynamoDS/Dynamo/pull/6062","id":130909047,"number":6062,"title":"Enable \"List.ShiftIndices\" to accept amounts greater than list length","user":{"login":"dimven","id":7148394,"avatar_url":"https://avatars.githubusercontent.com/u/7148394?v=3","gravatar_id":"","url":"https://api.github.com/users/dimven","html_url":"https://github.com/dimven","followers_url":"https://api.github.com/users/dimven/followers","following_url":"https://api.github.com/users/dimven/following{/other_user}","gists_url":"https://api.github.com/users/dimven/gists{/gist_id}","starred_url":"https://api.github.com/users/dimven/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dimven/subscriptions","organizations_url":"https://api.github.com/users/dimven/orgs","repos_url":"https://api.github.com/users/dimven/repos","events_url":"https://api.github.com/users/dimven/events{/privacy}","received_events_url":"https://api.github.com/users/dimven/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2016-02-03T04:33:19Z","updated_at":"2016-02-04T02:26:31Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/DynamoDS/Dynamo/pulls/6062","html_url":"https://github.com/DynamoDS/Dynamo/pull/6062","diff_url":"https://github.com/DynamoDS/Dynamo/pull/6062.diff","patch_url":"https://github.com/DynamoDS/Dynamo/pull/6062.patch"},"body":"### Purpose\r\nCurrently \"List.ShiftIndices\" borks if the amount of shifts is greater than the total list length.\r\n![2016-02-03_12-02-05](https://cloud.githubusercontent.com/assets/7148394/12772995/19781fe4-ca72-11e5-9873-41a9474e0338.png)\r\n![2016-02-03_11-58-46](https://cloud.githubusercontent.com/assets/7148394/12772994/19745bd4-ca72-11e5-8e51-0f9ef1df652e.png)\r\n\r\nWith this tiny change, we can remove this limitation:\r\n![2016-02-03_11-58-54](https://cloud.githubusercontent.com/assets/7148394/12773007/2ef1ecec-ca72-11e5-8c4d-9e429a4a5357.png)\r\n![2016-02-03_12-01-56](https://cloud.githubusercontent.com/assets/7148394/12773008/2efaa3e6-ca72-11e5-991b-2b8be158f8f2.png)\r\n\r\n### Declarations\r\n\r\n- [X] The code base is in a better state after this PR\r\n- [x] The level of testing this PR includes is appropriate\r\n- [x] All tests pass using the self-service CI.\r\n\r\n\r\n### Reviewers\r\n@aparajit-pratap @kronz \r\n\r\nAny chance you can fit this in 0.91 stable?"},"comment":{"url":"https://api.github.com/repos/DynamoDS/Dynamo/issues/comments/179580447","html_url":"https://github.com/DynamoDS/Dynamo/pull/6062#issuecomment-179580447","issue_url":"https://api.github.com/repos/DynamoDS/Dynamo/issues/6062","id":179580447,"user":{"login":"aparajit-pratap","id":5710686,"avatar_url":"https://avatars.githubusercontent.com/u/5710686?v=3","gravatar_id":"","url":"https://api.github.com/users/aparajit-pratap","html_url":"https://github.com/aparajit-pratap","followers_url":"https://api.github.com/users/aparajit-pratap/followers","following_url":"https://api.github.com/users/aparajit-pratap/following{/other_user}","gists_url":"https://api.github.com/users/aparajit-pratap/gists{/gist_id}","starred_url":"https://api.github.com/users/aparajit-pratap/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aparajit-pratap/subscriptions","organizations_url":"https://api.github.com/users/aparajit-pratap/orgs","repos_url":"https://api.github.com/users/aparajit-pratap/repos","events_url":"https://api.github.com/users/aparajit-pratap/events{/privacy}","received_events_url":"https://api.github.com/users/aparajit-pratap/received_events","type":"User","site_admin":false},"created_at":"2016-02-04T02:26:31Z","updated_at":"2016-02-04T02:26:31Z","body":"@dimven yes the test case looks good. Could you add the commit to the same PR?"}}