id,node_id,number,state,locked,title,user,body,created_at,updated_at,closed_at,merged_at,merge_commit_sha,assignee,milestone,draft,head,base,author_association,auto_merge,repo,url,merged_by 408238050,MDExOlB1bGxSZXF1ZXN0NDA4MjM4MDUw,3999,closed,0,Fix some code quality and bug-risk issues,40136154,"- Fix dangerous default argument - Remove unnecessary comprehension --- Find the other issues found here - [https://deepsource.io/gh/pnijhara/xarray/issues/?category=all](https://deepsource.io/gh/pnijhara/xarray/issues/?category=all) This PR also adds `.deepsource.toml` configuration file to run DeepSource analysis on the repo with. Upon enabling DeepSource, the analysis will run on every PR and commit to detect 560+ types of issues in the changes — including bug risks, anti-patterns, security vulnerabilities, etc. To enable DeepSource analysis after merging this PR, please follow these steps: 1. [Signup](https://deepsource.io/signup/) on DeepSource with your GitHub account and grant access to this repo. 2. Activate analysis on this repo [here](http://deepsource.io/gh/pydata/xarray) You can also look at the [docs](https://deepsource.io/docs/guides/quickstart.html) for more details. Do let me know if I can be of any help! ",2020-04-23T22:46:31Z,2020-05-05T02:21:12Z,2020-04-24T07:14:55Z,2020-04-24T07:14:55Z,37551da5ebc7861439ac3eefddefb534b76f2895,,,0,f835f1f295f86c1b394927a726ae5d141065f230,c788ee44008cdd65c8b6de40c737f1b28e173496,CONTRIBUTOR,,13221727,https://github.com/pydata/xarray/pull/3999, 408945898,MDExOlB1bGxSZXF1ZXN0NDA4OTQ1ODk4,4005,closed,0,Fix dangerous default argument,40136154," - [x] Closes #4002 - [ ] Tests added - [ ] Passes `isort -rc . && black . && mypy . && flake8` - [ ] Fully documented, including `whats-new.rst` for all changes and `api.rst` for new API ",2020-04-25T17:55:07Z,2020-04-25T17:58:24Z,2020-04-25T17:55:32Z,,dd3564caf2c08b8a340cb11a0e116ee38d921f95,,,0,8047c6f89bf98493ecdfb07b8c5384d1998c2740,33a66d6380c26a59923922ee11e8ffcf0b4f379f,CONTRIBUTOR,,13221727,https://github.com/pydata/xarray/pull/4005, 408947244,MDExOlB1bGxSZXF1ZXN0NDA4OTQ3MjQ0,4006,closed,0,fix dangerous default arguments,40136154," - [x] Closes #4002 - [ ] Passes `isort -rc . && black . && mypy . && flake8` - [ ] Fully documented, including `whats-new.rst` for all changes and `api.rst` for new API ",2020-04-25T18:05:11Z,2020-05-23T17:41:56Z,2020-05-23T17:30:51Z,2020-05-23T17:30:51Z,19b088636eb7d3f65ab7a1046ac672e0689371d8,,,0,b0e9cb17b531b756a9a0d4d1200b6c0e7d097e8f,484d1ce5ff8969b6ca6fa942b344379725f33b9c,CONTRIBUTOR,,13221727,https://github.com/pydata/xarray/pull/4006, 412931675,MDExOlB1bGxSZXF1ZXN0NDEyOTMxNjc1,4026,closed,0,chore: Remove unnecessary comprehension,40136154," - [x] Passes `isort -rc . && black . && mypy . && flake8` - [x] Fully documented, including `whats-new.rst` for all changes and `api.rst` for new API ",2020-05-04T12:49:41Z,2020-05-05T01:57:32Z,2020-05-05T01:57:31Z,2020-05-05T01:57:31Z,1b3c76863041d3265e5d011e68482944c447d78f,,,0,c71a77e0d23fb090f042df460ed4fad599dd17c7,6ce07249ca7eabc181b2b88a5723e66fc06036d4,CONTRIBUTOR,,13221727,https://github.com/pydata/xarray/pull/4026, 414255487,MDExOlB1bGxSZXF1ZXN0NDE0MjU1NDg3,4038,closed,0,Use literal syntax instead of function calls to create the data structure,40136154," - [x] Passes `isort -rc . && black . && mypy . && flake8` - [x] Fully documented, including `whats-new.rst` for all changes and `api.rst` for new API --- It is slower to call e.g. `list()` than using the empty literal, because the name list must be looked up in the global scope in case it has been rebound. Same for the other two types like `dict()` and `tuple()`.",2020-05-06T18:17:40Z,2020-05-06T22:55:40Z,2020-05-06T22:55:40Z,2020-05-06T22:55:40Z,0e43ba9c7080fc5f1076b0bccec09aeac1a3499b,,,0,04fb468e1b5d061a284db2c86eeaad9dc99ff6e0,0b6e22f6af71cc19d2e0575ecdf5c66109c314dd,CONTRIBUTOR,,13221727,https://github.com/pydata/xarray/pull/4038,