{"push_id":2411373220,"size":47,"distinct_size":1,"ref":"refs/heads/rust-language-server","head":"74212baea428fdfb8f5fd6e175aa7ced254489a3","before":"edc9562404a0a4732cdd2115ff2863c8996553f9","commits":[{"sha":"cec1a7188090ef761b71b236c0ebe074fc012e67","author":{"name":"micbou","email":"1927999686014308dead34c1cfcb9d8bd5271589@micbou.com"},"message":"Fix instructions for C#\n\nmono-devel should be installed instead of mono-xbuild.","distinct":false,"url":"https://api.github.com/repos/micbou/ycmd/commits/cec1a7188090ef761b71b236c0ebe074fc012e67"},{"sha":"da8873bb9e2c32ed74ea0a778f9730a130784b72","author":{"name":"micbou","email":"1927999686014308dead34c1cfcb9d8bd5271589@micbou.com"},"message":"Replace apt-get with apt","distinct":false,"url":"https://api.github.com/repos/micbou/ycmd/commits/da8873bb9e2c32ed74ea0a778f9730a130784b72"},{"sha":"be560dc5e190c66c6bafa3c64455d97be5bab798","author":{"name":"zzbot","email":"5e344a9fd57804fc7c6aecb84a3f2649d5f8e0d8@markovic.io"},"message":"Auto merge of #929 - micbou:fix-cs-instructions, r=puremourning\n\n[READY] Fix instructions for C#\n\n`mono-devel` should be installed instead of `mono-xbuild` for C# support.\n\nCloses https://github.com/Valloric/ycmd/issues/928.\n\n<!-- Reviewable:start -->\n---\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/valloric/ycmd/929)\n<!-- Reviewable:end -->","distinct":false,"url":"https://api.github.com/repos/micbou/ycmd/commits/be560dc5e190c66c6bafa3c64455d97be5bab798"},{"sha":"dcfbbb85691c8d27790c6729ca8ebeac2d13a3dd","author":{"name":"micbou","email":"1927999686014308dead34c1cfcb9d8bd5271589@micbou.com"},"message":"Add test where file contents change while polling messages","distinct":false,"url":"https://api.github.com/repos/micbou/ycmd/commits/dcfbbb85691c8d27790c6729ca8ebeac2d13a3dd"},{"sha":"79228e8e3a661e9e76e3f72c119b54c06109dbed","author":{"name":"micbou","email":"1927999686014308dead34c1cfcb9d8bd5271589@micbou.com"},"message":"Use server file contents if available for asynchronous diagnostics","distinct":false,"url":"https://api.github.com/repos/micbou/ycmd/commits/79228e8e3a661e9e76e3f72c119b54c06109dbed"},{"sha":"702c14edf8d6998b7bffccd379f526aec9c06bd7","author":{"name":"zzbot","email":"5e344a9fd57804fc7c6aecb84a3f2649d5f8e0d8@markovic.io"},"message":"Auto merge of #931 - micbou:server-file-state, r=puremourning\n\n[READY] Use file contents from server state if available when polling asynchronous diagnostics\n\nWhile I was playing with the Java completer, I noticed that matches from asynchronous diagnostics are sometimes incorrectly displayed:\n\n![file-contents-change-diagnostics-issue](https://user-images.githubusercontent.com/10026824/36354406-16398a80-14d4-11e8-838a-33b1998bd566.gif)\n\nThis is due to the file contents from the `/receive_messages` request not being up to date anymore when the language server are publishing new diagnostics. In the example above, when inserting the second `public String test;`, the diagnostic offsets are converted from UTF-16 to UTF-8 using the contents:\n```java\npackage com.youcompleteme;\n\npublic class Test {\n  public String test;\n}\n```\ninstead of\n```java\npackage com.youcompleteme;\n\npublic class Test {\n  public String test;\n  public String test;\n}\n```\n\nThe solution is to store new file contents in the `ServerFileState` object and use that contents (if available) when polling asynchronous diagnostics:\n\n![file-contents-change-diagnostics-fix](https://user-images.githubusercontent.com/10026824/36354449-f20a23e4-14d4-11e8-8c91-75f5bbd1ee05.gif)\n\nAdded a test that fails without this change.\n\n<!-- Reviewable:start -->\n---\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/valloric/ycmd/931)\n<!-- Reviewable:end -->","distinct":false,"url":"https://api.github.com/repos/micbou/ycmd/commits/702c14edf8d6998b7bffccd379f526aec9c06bd7"},{"sha":"20ca01e4ad28638cb9ff4f7b82e5feb60c07f42d","author":{"name":"micbou","email":"1927999686014308dead34c1cfcb9d8bd5271589@micbou.com"},"message":"Do not try to close stderr in Java completer\n\nSince stderr is redirected to a file in the Java completer, it is necessarily set to None and thus can't be closed.","distinct":false,"url":"https://api.github.com/repos/micbou/ycmd/commits/20ca01e4ad28638cb9ff4f7b82e5feb60c07f42d"},{"sha":"07332bb6ac929e327f9f9d959b30a0b2e7685d06","author":{"name":"zzbot","email":"5e344a9fd57804fc7c6aecb84a3f2649d5f8e0d8@markovic.io"},"message":"Auto merge of #932 - micbou:java-completer-stderr, r=puremourning\n\n[READY] Do not try to close stderr in Java completer\n\nSince `stderr` is redirected to a file in the Java completer, [it is necessarily set to `None`](https://docs.python.org/2.7/library/subprocess.html#subprocess.Popen.stderr) and thus can't be closed.\n\n<!-- Reviewable:start -->\n---\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/valloric/ycmd/932)\n<!-- Reviewable:end -->","distinct":false,"url":"https://api.github.com/repos/micbou/ycmd/commits/07332bb6ac929e327f9f9d959b30a0b2e7685d06"},{"sha":"756fb18e76ed0e67f30af9c6f6377556ec02a83b","author":{"name":"micbou","email":"1927999686014308dead34c1cfcb9d8bd5271589@micbou.com"},"message":"Do not check if server is running immediately after starting it","distinct":false,"url":"https://api.github.com/repos/micbou/ycmd/commits/756fb18e76ed0e67f30af9c6f6377556ec02a83b"},{"sha":"49763caf33dd2c0737a109e3622cc08c2accd966","author":{"name":"zzbot","email":"5e344a9fd57804fc7c6aecb84a3f2649d5f8e0d8@markovic.io"},"message":"Auto merge of #933 - micbou:java-completer-server-is-running, r=puremourning\n\n[READY] Do not check if server is running immediately after starting it\n\nGiven that `Popen` is returning immediately, the process will always be running at the `_ServerIsRunning` call even if it's going to crash. A simple way to confirm this is to comment the `-jar` argument in the command used to start jdt.ls: `_ServerIsRunning` still returns `True` in that scenario.\n\n<!-- Reviewable:start -->\n---\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/valloric/ycmd/933)\n<!-- Reviewable:end -->","distinct":false,"url":"https://api.github.com/repos/micbou/ycmd/commits/49763caf33dd2c0737a109e3622cc08c2accd966"},{"sha":"70aaabc0dcecbe804fd961a87ebec04ab008a8a3","author":{"name":"micbou","email":"1927999686014308dead34c1cfcb9d8bd5271589@micbou.com"},"message":"Fix error removing cfamily element if not present","distinct":false,"url":"https://api.github.com/repos/micbou/ycmd/commits/70aaabc0dcecbe804fd961a87ebec04ab008a8a3"},{"sha":"bec70588a588517e06973efddc0e444a21f1ac42","author":{"name":"zzbot","email":"5e344a9fd57804fc7c6aecb84a3f2649d5f8e0d8@markovic.io"},"message":"Auto merge of #934 - micbou:run-tests-fix-removing-cfamily, r=bstaletic\n\n[READY] Fix error removing cfamily element if not present\n\nRunning the tests with `USE_CLANG_COMPLETER=false ./run_tests.py --no-completers cfamily` raise the following error:\n```\nTraceback (most recent call last):\n  File \"./run_tests.py\", line 239, in <module>\n    Main()\n  File \"./run_tests.py\", line 227, in Main\n    parsed_args, nosetests_args = ParseArguments()\n  File \"./run_tests.py\", line 136, in ParseArguments\n    parsed_args.completers = FixupCompleters( parsed_args )\n  File \"./run_tests.py\", line 157, in FixupCompleters\n    completers.remove( 'cfamily' )\nKeyError: 'cfamily'\n```\nUse `discard` instead of `remove` to remove an element from the set of completers.\n\n<!-- Reviewable:start -->\n---\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/valloric/ycmd/934)\n<!-- Reviewable:end -->","distinct":false,"url":"https://api.github.com/repos/micbou/ycmd/commits/bec70588a588517e06973efddc0e444a21f1ac42"},{"sha":"acea2b3accfc0c419b682e5ef1ca117e5d8f8d3a","author":{"name":"Boris Staletic","email":"047bd7d1d2e62e5c9d7de617054590b4b979f77c@iten.rs"},"message":"Add clang-cl support","distinct":false,"url":"https://api.github.com/repos/micbou/ycmd/commits/acea2b3accfc0c419b682e5ef1ca117e5d8f8d3a"},{"sha":"3be179974ae55238cd5db5e22c804b45c169f65b","author":{"name":"zzbot","email":"5e344a9fd57804fc7c6aecb84a3f2649d5f8e0d8@markovic.io"},"message":"Auto merge of #936 - bstaletic:clang-cl_support, r=micbou\n\n[READY] Add clang-cl support\n\nAccording to https://github.com/Valloric/ycmd/pull/789#issuecomment-367151834, #789 didn't include support for `clang-cl` without `--driver-mode=cl` specified.\n\nI did not add `clang-cl` to the C++ regex. Since there is no `clang-cl++` I think we should not touch language flags and let clang decide if it's C or C++.\n\n<!-- Reviewable:start -->\n---\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/valloric/ycmd/936)\n<!-- Reviewable:end -->","distinct":false,"url":"https://api.github.com/repos/micbou/ycmd/commits/3be179974ae55238cd5db5e22c804b45c169f65b"},{"sha":"6c9cd94c072276c0baabd053fcb8f968eb685db4","author":{"name":"micbou","email":"1927999686014308dead34c1cfcb9d8bd5271589@micbou.com"},"message":"Skip stray Windows paths in flags","distinct":false,"url":"https://api.github.com/repos/micbou/ycmd/commits/6c9cd94c072276c0baabd053fcb8f968eb685db4"},{"sha":"f260d2e1fcddd33fb319abfd277fe169108750e3","author":{"name":"zzbot","email":"5e344a9fd57804fc7c6aecb84a3f2649d5f8e0d8@markovic.io"},"message":"Auto merge of #938 - micbou:stray-windows-paths, r=bstaletic\n\n[READY] Skip stray Windows paths in flags\n\nWhen using a compilation database, the flags generally end with `-c /path/to/file` where `/path/to/file` is the compiled file. Libclang chokes on this if we don't remove the path. We have some logic to handle that case where, in addition to several conditions, we detect that the flag is a path if it contains the `/` separator. However, this doesn't take care of the case where the flag is a Windows path with `\\` as separators.\n\nFixes #937.\n\n<!-- Reviewable:start -->\n---\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/valloric/ycmd/938)\n<!-- Reviewable:end -->","distinct":false,"url":"https://api.github.com/repos/micbou/ycmd/commits/f260d2e1fcddd33fb319abfd277fe169108750e3"},{"sha":"90f2b1f827a1e2ea1d8e9e1023b999019dea505b","author":{"name":"micbou","email":"1927999686014308dead34c1cfcb9d8bd5271589@micbou.com"},"message":"Lower minimum Python version requirements on Travis and CircleCI","distinct":false,"url":"https://api.github.com/repos/micbou/ycmd/commits/90f2b1f827a1e2ea1d8e9e1023b999019dea505b"},{"sha":"780ba9e0adce254525890e6b07eda4ec0db510ab","author":{"name":"micbou","email":"1927999686014308dead34c1cfcb9d8bd5271589@micbou.com"},"message":"Download libclang from Bintray\n\nRename make_llvm_package script to update_clang and improve it with the\nfollowing changes:\n - automatically update the Clang headers;\n - download Clang archives for FreeBSD, AArch64, and armv7a;\n - download the license only once;\n - do not hard code the version in the files to copy for Linux;\n - use regedit to find 7-Zip on Windows;\n - silence 7-Zip output;\n - raise an error if one of the files in the downloaded archives does not\n   exist;\n - use HTTPS for the LLVM URL;\n - minor style changes.","distinct":false,"url":"https://api.github.com/repos/micbou/ycmd/commits/780ba9e0adce254525890e6b07eda4ec0db510ab"},{"sha":"2d36c76e059b782707ff62ac01ddee7514259593","author":{"name":"zzbot","email":"5e344a9fd57804fc7c6aecb84a3f2649d5f8e0d8@markovic.io"},"message":"Auto merge of #940 - micbou:lower-minimum-python-version-requirements, r=bstaletic\n\n[READY] Lower minimum Python version requirements on Travis and CircleCI\n\nAs discussed on Gitter some time ago, we should try to test for the oldest version of Python that we are supposed to support on Linux and macOS (e.g. RHEL 7 has Python 2.7.5 installed by default so if we test for Python 2.7.14, we are not sure it's going to work on that platform).\n\n<!-- Reviewable:start -->\n---\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/valloric/ycmd/940)\n<!-- Reviewable:end -->","distinct":false,"url":"https://api.github.com/repos/micbou/ycmd/commits/2d36c76e059b782707ff62ac01ddee7514259593"},{"sha":"56662ec92bb34e85f08456dfb07ce454a1e0c642","author":{"name":"micbou","email":"1927999686014308dead34c1cfcb9d8bd5271589@micbou.com"},"message":"Drop Vagrant setup","distinct":false,"url":"https://api.github.com/repos/micbou/ycmd/commits/56662ec92bb34e85f08456dfb07ce454a1e0c642"}]}
{"actor":{"display_login":"micbou"}}