html_url,issue_url,id,node_id,user,created_at,updated_at,author_association,body,reactions,performed_via_github_app,issue
https://github.com/pydata/xarray/pull/5845#issuecomment-959835614,https://api.github.com/repos/pydata/xarray/issues/5845,959835614,IC_kwDOAMm_X845Ne3e,170287,2021-11-03T19:04:45Z,2021-11-03T19:04:45Z,CONTRIBUTOR,"> Thanks @marscher This is a great change.
>
> I see this is your first PR. Welcome to Xarray!
Thanks for the warm welcome.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1020407925
https://github.com/pydata/xarray/pull/5845#issuecomment-956286144,https://api.github.com/repos/pydata/xarray/issues/5845,956286144,IC_kwDOAMm_X844_8TA,2448579,2021-11-01T14:33:19Z,2021-11-01T14:33:19Z,MEMBER,"Thanks @marscher This is a great change.
I see this is your first PR. Welcome to Xarray!","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1020407925
https://github.com/pydata/xarray/pull/5845#issuecomment-938114133,https://api.github.com/repos/pydata/xarray/issues/5845,938114133,IC_kwDOAMm_X8436nxV,41898282,2021-10-07T20:02:55Z,2021-10-29T17:40:36Z,CONTRIBUTOR,"## Unit Test Results
0 files 0 suites 0s [:stopwatch:](https://github.com/EnricoMi/publish-unit-test-result-action/blob/v1.20/README.md#the-symbols ""duration of all tests"")
0 tests 0 [:heavy_check_mark:](https://github.com/EnricoMi/publish-unit-test-result-action/blob/v1.20/README.md#the-symbols ""passed tests"") 0 [:zzz:](https://github.com/EnricoMi/publish-unit-test-result-action/blob/v1.20/README.md#the-symbols ""skipped / disabled tests"") 0 [:x:](https://github.com/EnricoMi/publish-unit-test-result-action/blob/v1.20/README.md#the-symbols ""failed tests"")
Results for commit 28c84858.
:recycle: This comment has been updated with latest results.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1020407925
https://github.com/pydata/xarray/pull/5845#issuecomment-938111206,https://api.github.com/repos/pydata/xarray/issues/5845,938111206,IC_kwDOAMm_X8436nDm,24736507,2021-10-07T19:58:42Z,2021-10-29T17:39:51Z,NONE,"Hello @marscher! Thanks for updating this PR. We checked the lines you've touched for [PEP 8](https://www.python.org/dev/peps/pep-0008) issues, and found:
There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:
##### Comment last updated at 2021-10-29 17:39:51 UTC","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1020407925
https://github.com/pydata/xarray/pull/5845#issuecomment-954886300,https://api.github.com/repos/pydata/xarray/issues/5845,954886300,IC_kwDOAMm_X8446mic,2448579,2021-10-29T16:35:43Z,2021-10-29T16:35:43Z,MEMBER,"Test failure is real
```
xarray/__init__.py:34: error: Name ""_version"" already defined (possibly by an import) [no-redef]
Found 1 error in 1 file (checked 145 source files)
```
I think we just revert the last commit and merge...","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1020407925
https://github.com/pydata/xarray/pull/5845#issuecomment-954595452,https://api.github.com/repos/pydata/xarray/issues/5845,954595452,IC_kwDOAMm_X8445fh8,170287,2021-10-29T09:32:34Z,2021-10-29T09:32:34Z,CONTRIBUTOR,All suggestions of yours are now included and CI is green. Thank you all very much for your review and suggestions.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1020407925
https://github.com/pydata/xarray/pull/5845#issuecomment-954246645,https://api.github.com/repos/pydata/xarray/issues/5845,954246645,IC_kwDOAMm_X8444KX1,14808389,2021-10-28T21:45:31Z,2021-10-28T21:47:35Z,MEMBER,"thanks for the PR, @marscher, and especially for being patient with us/me.
if the CI still passes without the explicit mention of `setuptools` in the CI environments this should be ready for merging. It would probably be good to review the change to the dependency policy, though (do we need to list `setuptools` even if we only depend on it at build time?)","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1020407925
https://github.com/pydata/xarray/pull/5845#issuecomment-953773679,https://api.github.com/repos/pydata/xarray/issues/5845,953773679,IC_kwDOAMm_X8442W5v,14808389,2021-10-28T11:55:24Z,2021-10-28T11:55:24Z,MEMBER,"the CI passes, too. Now we just need an entry in `whats-new.rst` saying that the runtime dependency on `setuptools` was dropped, and we might also want to remove the explicit mention of `setuptools` in the CI files","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1020407925
https://github.com/pydata/xarray/pull/5845#issuecomment-953722459,https://api.github.com/repos/pydata/xarray/issues/5845,953722459,IC_kwDOAMm_X8442KZb,170287,2021-10-28T10:37:30Z,2021-10-28T10:37:30Z,CONTRIBUTOR,I installed the precommit hook and everything passed locally.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1020407925
https://github.com/pydata/xarray/pull/5845#issuecomment-953259849,https://api.github.com/repos/pydata/xarray/issues/5845,953259849,IC_kwDOAMm_X8440ZdJ,170287,2021-10-27T19:53:32Z,2021-10-27T19:53:32Z,CONTRIBUTOR,I wonder if there are any tests for the backend entry points or if tests are just being skipped if the mandatory backends are unavailble (due to a bug introduced by switching from pkg_resources). I have like 5600 skipped tests when I naively invoke pytest. ,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1020407925
https://github.com/pydata/xarray/pull/5845#issuecomment-953085452,https://api.github.com/repos/pydata/xarray/issues/5845,953085452,IC_kwDOAMm_X844zu4M,14808389,2021-10-27T16:15:43Z,2021-10-27T16:45:37Z,MEMBER,"thanks, @dopplershift. We agreed on doing that in the last meeting, and to then drop the new dependency together with `python=3.7` in December. That means that if we can figure out the remaining issues we can include this in the next release (which will be very soon!)","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1020407925
https://github.com/pydata/xarray/pull/5845#issuecomment-951404731,https://api.github.com/repos/pydata/xarray/issues/5845,951404731,IC_kwDOAMm_X844tUi7,221526,2021-10-25T23:10:25Z,2021-10-25T23:10:25Z,CONTRIBUTOR,"You cannot use selectors with noarch conda packages. Full stop.
For conda-forge, it's perfectly fine to just unconditionally depend on the `importlib_metadata` backport, especially with the implementation done like here where you use the std lib version and fall back.","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1020407925
https://github.com/pydata/xarray/pull/5845#issuecomment-946419161,https://api.github.com/repos/pydata/xarray/issues/5845,946419161,IC_kwDOAMm_X844aTXZ,170287,2021-10-19T06:56:22Z,2021-10-19T06:56:22Z,CONTRIBUTOR,"If one installs local packages with conda, all dependencies are ignored (--use-local flag). I have not yet figured out, why this is the case due to a lack of time.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1020407925
https://github.com/pydata/xarray/pull/5845#issuecomment-940127873,https://api.github.com/repos/pydata/xarray/issues/5845,940127873,IC_kwDOAMm_X844CTaB,14808389,2021-10-11T15:23:06Z,2021-10-11T15:24:43Z,MEMBER,"I might be confusing something, but I thought preprocessing selectors were executed at build time? So if you build the `noarch: python` package in an environment that has `python=3.7` you will keep the `importlib_metadata` while in a different one with `python=3.8` it will be removed?
The easiest way to figure this out would be to build in a `py38` environment, install in a `py37` environment and then try to import it (i.e. trigger the code path that uses `importlib.metadata`). If that works, then it would be fine to merge now.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1020407925
https://github.com/pydata/xarray/pull/5845#issuecomment-939985659,https://api.github.com/repos/pydata/xarray/issues/5845,939985659,IC_kwDOAMm_X844Bwr7,170287,2021-10-11T12:29:19Z,2021-10-11T12:38:01Z,CONTRIBUTOR,"I don't think that noarch packages are restricted to python version selectors any more. I've tested this locally with a dummy recipe like this:
```yaml
package:
name: {{ name|lower }}
version: {{ version }}
source:
path: .
build:
noarch: python
number: 0
script: ""{{ PYTHON }} -m pip install . -vv""
requirements:
host:
- python
- pip
run:
- python
- importlib_metadata # [ py < 38]
```
This creates a noarch package as expected with a recent conda/build version.
This means you can merge this and still use a noarch package on Conda-forge, which will conditionally install the backport importlib-metadata package for older Python versions.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1020407925
https://github.com/pydata/xarray/pull/5845#issuecomment-939371374,https://api.github.com/repos/pydata/xarray/issues/5845,939371374,IC_kwDOAMm_X843_atu,14808389,2021-10-09T22:41:15Z,2021-10-10T16:40:21Z,MEMBER,"> is there any reason not to merge this?
the issue is that with `conda` we can either always install `importlib_metadata` or not at all: `conda` doesn't support `noarch_python` packages with preprocessing selectors (see the comments around https://github.com/pydata/xarray/issues/4295#issuecomment-667657310 for more discussion).
`importlib.metadata` is included in `python>=3.8`, which we can officially require from end of December this year.
All of this means that if we want to merge this now we have to decide between not releasing until we can drop `python=3.7` or always installing `importlib_metadata` (which is what the `pint` feedstock does). However, I guess the cleanest way would be to wait until we can require `python>=3.8`.","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1020407925
https://github.com/pydata/xarray/pull/5845#issuecomment-939368646,https://api.github.com/repos/pydata/xarray/issues/5845,939368646,IC_kwDOAMm_X843_aDG,5635139,2021-10-09T22:11:31Z,2021-10-09T22:11:31Z,MEMBER,"Thanks for the PR @marscher
I don't know this area well — assuming this cuts the import time down considerably, is there any reason not to merge this?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1020407925