{"push_id":1065147723,"size":47,"distinct_size":46,"ref":"refs/heads/master","head":"e331efa95cd1f444c825ee16e3ed2009b0cafe45","before":"a9237d66f29c6077f81b09d30887f73b8ac5d1cc","commits":[{"sha":"b33d4783bce48a36e2400c54b1371fe68473596a","author":{"email":"b1c1d8736f20db3fb6c1c66bb1455ed43909f0d8@arrdem.com","name":"Reid 'arrdem' McKenzie"},"message":"Include cider-nrepl anyway when clj is excluded\n\nThis patch allows the cider-nrepl middleware to be injected into a\nproject which does not depend on `org.clojure/clojure`, if and only if\nthe user has explicitly provided `:exclusions [... org.clojure/clojure]`\nin the `project.clj` file. This allows users who know what they're doing\nto take on the responsibility of providing an appropriate build of\nClojure and thus use CIDER with forks albeit with a warning.","distinct":false,"url":"https://api.github.com/repos/arrdem/cider-nrepl/commits/b33d4783bce48a36e2400c54b1371fe68473596a"},{"sha":"4ecb40c6028a399089ded66b670635c522d18b91","author":{"email":"219199d75e607b143e770035d8a9199c2af66000@gmail.com","name":"Bozhidar Batsov"},"message":"Merge pull request #316 from arrdem/feature/excluded-clojure\n\nInclude cider-nrepl anyway when clj is excluded","distinct":true,"url":"https://api.github.com/repos/arrdem/cider-nrepl/commits/4ecb40c6028a399089ded66b670635c522d18b91"},{"sha":"85994cbd1b2a8430b997744f01d28f26ec1b94ea","author":{"email":"89201684f0348f1e0eae195a64fba6f13951640d@gmail.com","name":"sanjayl"},"message":"Fix Javadoc URLs for Java 1.8\n\nAn issue where the URLs for Javadoc in Java 1.8 has a different format\nthan prior Java version, namely, parenthesis surrounding argument lists\nhave been changed to dashes. Added tests as well.","distinct":true,"url":"https://api.github.com/repos/arrdem/cider-nrepl/commits/85994cbd1b2a8430b997744f01d28f26ec1b94ea"},{"sha":"2fa36d1990316744dc732415c7516cf5bfcf6f0f","author":{"email":"219199d75e607b143e770035d8a9199c2af66000@gmail.com","name":"Bozhidar Batsov"},"message":"Merge pull request #315 from sanjayl/javadoc-fix\n\nFix Javadoc URLs for Java 1.8","distinct":true,"url":"https://api.github.com/repos/arrdem/cider-nrepl/commits/2fa36d1990316744dc732415c7516cf5bfcf6f0f"},{"sha":"32774140e7ce1139555948dae93ad8b2ca2c258f","author":{"email":"89201684f0348f1e0eae195a64fba6f13951640d@gmail.com","name":"sanjayl"},"message":"Expand Info middleware tests\n\nAdded tests to exercise the Javadoc URL issue [Fix #308].\n\nAdded test to see if the *remote-javadocs* functionality works (same fix as\nabove).\n\nChanged some of the testing targets to internal test classes/methods\ninstead of testing against code outside of our control when possible. As\na result, added a java source tree under the test directory and added\nthis to the test-clj profile.\n\nBrings coverage back up to 100%.","distinct":true,"url":"https://api.github.com/repos/arrdem/cider-nrepl/commits/32774140e7ce1139555948dae93ad8b2ca2c258f"},{"sha":"30982c1047f55f30694c31b11967db0decb1c63e","author":{"email":"2668f6386f9fb08ef1611e876f0a60a4211c12b2@gmail.com","name":"Chris Perkins"},"message":"Debugger: make skip-breaks depend on current code\n\nPreviously, the var `*skip-breaks*` was storing a mode (:deeper,\n:before, etc.) and a sequence of coordinates. However, it did not store\nany infomation indicating what code the coordinates refered to.\n\nThis would lead to strange behavior when a an :out or :here operation\nwould skip over code that called another instrumented function. The\ndebugger would treat the coordinates in *skip-breaks* as if they\nreferred to the code of the called function, and would seem (to the\nuser) to skip randomly over some or all of the function.\n\nThe fix:\n\n1) Store *skip-breaks* as a map containing :mode, :coor, and :code,\nwhere :code is the string of code that the :out or :here operation was\nperformed on.\n\n2) Add logic to `skip-breaks?` to only skip if the to-be-skipped code\nand the currently-being-debugged code are the same.","distinct":true,"url":"https://api.github.com/repos/arrdem/cider-nrepl/commits/30982c1047f55f30694c31b11967db0decb1c63e"},{"sha":"96897350fb3abc624a49cc7a3c94b0c283eb0470","author":{"email":"2668f6386f9fb08ef1611e876f0a60a4211c12b2@gmail.com","name":"Chris Perkins"},"message":"Remove unnecessary binding of *skip-breaks*","distinct":true,"url":"https://api.github.com/repos/arrdem/cider-nrepl/commits/96897350fb3abc624a49cc7a3c94b0c283eb0470"},{"sha":"f98f67712df6a343dff21b3325f51af37e1ed73f","author":{"email":"1a52aee295acb1fce976c0a020f5563f49f05147@users.noreply.github.com","name":"Artur Malabarba"},"message":"Merge pull request #317 from sanjayl/redo-info\n\nExpand Info middleware tests","distinct":true,"url":"https://api.github.com/repos/arrdem/cider-nrepl/commits/f98f67712df6a343dff21b3325f51af37e1ed73f"},{"sha":"4460c82d8c3b5455a1f4a7769d09c3fb4d212be4","author":{"email":"1a52aee295acb1fce976c0a020f5563f49f05147@users.noreply.github.com","name":"Artur Malabarba"},"message":"Merge pull request #311 from grammati/skip-breaks-fix\n\nDebugger: make skip-breaks depend on current code","distinct":true,"url":"https://api.github.com/repos/arrdem/cider-nrepl/commits/4460c82d8c3b5455a1f4a7769d09c3fb4d212be4"},{"sha":"026d2e6e661ae21589cd49d623efe72ef9e702eb","author":{"email":"89201684f0348f1e0eae195a64fba6f13951640d@gmail.com","name":"sanjayl"},"message":"Fix inspector type errors and improve tests\n\nA couple of small bugs/inconsistencies in the inspector middleware.\n\n1 - Setting the page size on the fly while using the inspector was not\nworking. Emacs was sending an integer to Cider, which was expecting a\nstring, causing a type conversion error (the tests were sending a\nstring, which is why it wasn't caught before). Also, the user can pass\nin negative integers or a zero which would break the system, so added a\nsanity check.\n\n2 - Similarly, the push op expects a stringified integer in the :idx\nslot. The elisp code was converting the supplied integer to a string\nprior to sending it over the wire, so there was no error in the code,\nbut this has been changed so that the push op requires an integer and\nthe Cider Elisp code has been changed so that it will also send an\ninteger instead of unnecessarily converting it to a string. Please note\nthat if your Cider code is not as up to date as your Cider-nREPL code,\nthen the push function will not work.\n\nAlso added error handling try/catch blocks around all the messaging\noperations. Added tests to exercise these.","distinct":true,"url":"https://api.github.com/repos/arrdem/cider-nrepl/commits/026d2e6e661ae21589cd49d623efe72ef9e702eb"},{"sha":"61e470e739172263e510f18e7655584e065cb19c","author":{"email":"219199d75e607b143e770035d8a9199c2af66000@gmail.com","name":"Bozhidar Batsov"},"message":"Merge pull request #318 from sanjayl/inspect-coverage\n\nFix small inspector errors and improve tests","distinct":true,"url":"https://api.github.com/repos/arrdem/cider-nrepl/commits/61e470e739172263e510f18e7655584e065cb19c"},{"sha":"58eed3888b117c3eea24daf4040c0eca80edb80f","author":{"email":"89201684f0348f1e0eae195a64fba6f13951640d@gmail.com","name":"sanjayl"},"message":"Add error checking to ns middleware and add tests","distinct":true,"url":"https://api.github.com/repos/arrdem/cider-nrepl/commits/58eed3888b117c3eea24daf4040c0eca80edb80f"},{"sha":"7862254a412f8f7a5d44e341068d72e6671d4386","author":{"email":"219199d75e607b143e770035d8a9199c2af66000@gmail.com","name":"Bozhidar Batsov"},"message":"Merge pull request #320 from sanjayl/ns-coverage\n\nAdd error checking to ns middleware and add tests","distinct":true,"url":"https://api.github.com/repos/arrdem/cider-nrepl/commits/7862254a412f8f7a5d44e341068d72e6671d4386"},{"sha":"441d1fcd70ed70f6d0523a9e4081e1718a200135","author":{"email":"89201684f0348f1e0eae195a64fba6f13951640d@gmail.com","name":"sanjayl"},"message":"Simplify code, add coverage tests\n\nThe prior code had a difficult to test code block in `resolve-pprint-fn`\nas well as a small piece of round about code in `wrap-pprint` , so these\nfunctions were streamlined to help with testing/understanding. Behavior\nis slightly changed since the resolve-pprint-fn will now throw different\nexceptions based on the root cause of the exception (no such namespace\nin classpath, no such var in an existing namespace, etc. This required a\nslight change in the format-middleware tests). Otherwise, the\nfunctionality of the code should be the same.\n\nAdded coverage tests to get to 100%. Error handling is actually done\nelsewhere in the middleware stack (within the `eval` op).","distinct":true,"url":"https://api.github.com/repos/arrdem/cider-nrepl/commits/441d1fcd70ed70f6d0523a9e4081e1718a200135"},{"sha":"83c06ed8bbec8f1788000aac58f1cb8acf9ef89d","author":{"email":"2f2eda765b3bf18fb86627f973cc113f144d69d9@gmail.com","name":"Artur Malabarba"},"message":"[Fix #322] Print to original-out, not #'original-out","distinct":true,"url":"https://api.github.com/repos/arrdem/cider-nrepl/commits/83c06ed8bbec8f1788000aac58f1cb8acf9ef89d"},{"sha":"319c1029e2eafdda752acb90ad647cfeb6ee7a71","author":{"email":"219199d75e607b143e770035d8a9199c2af66000@gmail.com","name":"Bozhidar Batsov"},"message":"Merge pull request #321 from sanjayl/pprint-fix\n\nImprove pprint middleware tests & simplify code","distinct":true,"url":"https://api.github.com/repos/arrdem/cider-nrepl/commits/319c1029e2eafdda752acb90ad647cfeb6ee7a71"},{"sha":"86f6685a6ff94ae6f1664fbd22cb90965391f667","author":{"email":"2f2eda765b3bf18fb86627f973cc113f144d69d9@gmail.com","name":"Artur Malabarba"},"message":"Temporarily mark the openjdk7 builds as allow-failure","distinct":true,"url":"https://api.github.com/repos/arrdem/cider-nrepl/commits/86f6685a6ff94ae6f1664fbd22cb90965391f667"},{"sha":"a06ceade05cbaa0295a980638a352a29fbde80a3","author":{"email":"2668f6386f9fb08ef1611e876f0a60a4211c12b2@gmail.com","name":"Chris Perkins"},"message":"Add optional force parameter to :out and :here ops\n\nWhen an `:out` or `:here` operation jumps past calls to other\ninstrumented functions, the debugger will normally stop in those\nfunctions.\n\nPassing a truthy `force` parameter to one of these ops will cause the\ndebugger to skip all breaks outside of the currently debugged form,\neffecively forcing it to immediately step `:out` or `:here`.","distinct":true,"url":"https://api.github.com/repos/arrdem/cider-nrepl/commits/a06ceade05cbaa0295a980638a352a29fbde80a3"},{"sha":"7bc7843add1a6b83554a0750709e12658fdd2dff","author":{"email":"27dfa702b5736100b21d1436d073d499e72009ce@batsov.com","name":"Bozhidar Batsov"},"message":"[clojure-emacs/cider#1563] Handle explicitly invalid regular expressions","distinct":true,"url":"https://api.github.com/repos/arrdem/cider-nrepl/commits/7bc7843add1a6b83554a0750709e12658fdd2dff"},{"sha":"39f40b6a0101ffa18de9a1a21cd106a8de77a94b","author":{"email":"2f2eda765b3bf18fb86627f973cc113f144d69d9@gmail.com","name":"Artur Malabarba"},"message":"[Fix #1386] Reset debugger-message if the connection is closed","distinct":true,"url":"https://api.github.com/repos/arrdem/cider-nrepl/commits/39f40b6a0101ffa18de9a1a21cd106a8de77a94b"}]}