pull_requests: 335085538
This data as json
| 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 |
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 335085538 | MDExOlB1bGxSZXF1ZXN0MzM1MDg1NTM4 | 3475 | closed | 0 | drop_vars; deprecate drop for variables | 5635139 | <!-- Feel free to remove check-list items aren't relevant to your change --> Introduces `drop_vars`, and deprecates using `drop` for variables. `drop` is widely used for the deprecated case, so this is a fairly wide blast radius. It's more churn than is ideal, but I do think it's a much better API. This is ready for review, though I'm sure I'm missed references in the docs etc (took my peak regex skills to find/replace only the deprecated usages!) Originally discussed [here](https://github.com/pydata/xarray/pull/3460) - [x] Tests added - [x] Passes `black . && mypy . && flake8` - [x] Fully documented, including `whats-new.rst` for all changes and `api.rst` for new API | 2019-10-31T18:46:48Z | 2019-11-07T23:20:40Z | 2019-11-07T20:13:51Z | 2019-11-07T20:13:51Z | 0e8debfe28286b5fe1f3d27e8dcc8466a62aca6d | 0 | db4b2a39cb252706dbfcfa7bd235468aaadaaec2 | 4dce93f134e8296ea730104b46ce3372b90304ac | MEMBER | 13221727 | https://github.com/pydata/xarray/pull/3475 |
Links from other tables
- 0 rows from pull_requests_id in labels_pull_requests