Merge branch 'main' into mapblock-fix

* main:
  Replace markdown issue templates with issue forms (pydata#6119)
  Remove lock kwarg (pydata#5912)
  Remove pre-commit GHA workflow (pydata#6120)
  is_dask_collection: micro optimization (pydata#6107)

Author: dcherian

.github/ISSUE_TEMPLATE/bug-report.md

@@ -0,0 +1,61 @@
+name: Bug Report
+description: File a bug report to help us improve
+title: '[Bug]: '
+labels: [bug, 'needs triage']
+assignees: []
+body:
+  - type: textarea
+    id: what-happened
+    attributes:
+      label: What happened?
+      description: |
+        Thanks for reporting a bug! Please describe what you were trying to get done.
+        Tell us what happened, what went wrong.
+    validations:
+      required: true
+
+  - type: textarea
+    id: what-did-you-expect-to-happen
+    attributes:
+      label: What did you expect to happen?
+      description: |
+        Describe what you expected to happen.
+    validations:
+      required: false
+
+  - type: textarea
+    id: sample-code
+    attributes:
+      label: Minimal Complete Verifiable Example
+      description: |
+        Minimal, self-contained copy-pastable example that generates the issue if possible. Please be concise with code posted. See guidelines below on how to provide a good bug report:
+
+        - [Minimal Complete Verifiable Examples](https://stackoverflow.com/help/mcve)
+        - [Craft Minimal Bug Reports](http://matthewrocklin.com/blog/work/2018/02/28/minimal-bug-reports)
+
+        Bug reports that follow these guidelines are easier to diagnose, and so are often handled much more quickly.
+        This will be automatically formatted into code, so no need for markdown backticks.
+      render: python
+
+  - type: textarea
+    id: log-output
+    attributes:
+      label: Relevant log output
+      description: Please copy and paste any relevant output. This will be automatically formatted into code, so no need for markdown backticks.
+      render: python
+
+  - type: textarea
+    id: extra
+    attributes:
+      label: Anything else we need to know?
+      description: |
+        Please describe any other information you want to share.
+
+  - type: textarea
+    id: show-versions
+    attributes:
+     label: Environment
+     description: |
+       Paste the output of `xr.show_versions()` here
+    validations:
+      required: true

.github/ISSUE_TEMPLATE/bugreport.yml

@@ -0,0 +1,37 @@
+name: Feature Request
+description: Suggest an idea for xarray
+title: '[FEATURE]: '
+labels: [enhancement]
+assignees: []
+body:
+  - type: textarea
+    id: description
+    attributes:
+      label: Is your feature request related to a problem?
+      description: |
+        Please do a quick search of existing issues to make sure that this has not been asked before.
+        Please provide a clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
+    validations:
+      required: true
+  - type: textarea
+    id: solution
+    attributes:
+      label: Describe the solution you'd like
+      description: |
+        A clear and concise description of what you want to happen.
+  - type: textarea
+    id: alternatives
+    attributes:
+      label: Describe alternatives you've considered
+      description: |
+        A clear and concise description of any alternative solutions or features you've considered.
+    validations:
+      required: false
+  - type: textarea
+    id: additional-context
+    attributes:
+      label: Additional context
+      description: |
+        Add any other context about the feature request here.
+    validations:
+      required: false

.github/ISSUE_TEMPLATE/feature-request.md

@@ -2,6 +2,5 @@
 
 - [ ] Closes #xxxx
 - [ ] Tests added
-- [ ] Passes `pre-commit run --all-files`
 - [ ] User visible changes (including notable bug fixes) are documented in `whats-new.rst`
 - [ ] New functions/methods are listed in `api.rst`

.github/ISSUE_TEMPLATE/newfeature.yml

@@ -31,6 +31,8 @@ Breaking changes
 
 Deprecations
 ~~~~~~~~~~~~
+- Removed the lock kwarg from the zarr and pydap backends, completing the deprecation cycle started in :issue:`5256`.
+  By `Tom Nicholas <https://github.com/TomNicholas>`_.
 
 
 Bug fixes

.github/PULL_REQUEST_TEMPLATE.md

@@ -1,5 +1,3 @@
-import warnings
-
 import numpy as np
 
 from ..core import indexing
@@ -126,15 +124,7 @@ def open_dataset(
         use_cftime=None,
         decode_timedelta=None,
         session=None,
-        lock=None,
     ):
-        # TODO remove after v0.19
-        if lock is not None:
-            warnings.warn(
-                "The kwarg 'lock' has been deprecated for this backend, and is now "
-                "ignored. In the future passing lock will raise an error.",
-                DeprecationWarning,
-            )
 
         store = PydapDataStore.open(
             filename_or_obj,

.github/workflows/ci-pre-commit.yml

@@ -810,15 +810,7 @@ def open_dataset(
         chunk_store=None,
         storage_options=None,
         stacklevel=3,
-        lock=None,
     ):
-        # TODO remove after v0.19
-        if lock is not None:
-            warnings.warn(
-                "The kwarg 'lock' has been deprecated for this backend, and is now "
-                "ignored. In the future passing lock will raise an error.",
-                DeprecationWarning,
-            )
 
         filename_or_obj = _normalize_path(filename_or_obj)
         store = ZarrStore.open_group(

doc/whats-new.rst

@@ -43,8 +43,19 @@ def __init__(self, mod):
         self.available = duck_array_module is not None
 
 
+dsk = DuckArrayModule("dask")
+dask_version = dsk.version
+dask_array_type = dsk.type
+
+sp = DuckArrayModule("sparse")
+sparse_array_type = sp.type
+sparse_version = sp.version
+
+cupy_array_type = DuckArrayModule("cupy").type
+
+
 def is_dask_collection(x):
-    if DuckArrayModule("dask").available:
+    if dsk.available:
         from dask.base import is_dask_collection
 
         return is_dask_collection(x)
@@ -54,14 +65,3 @@ def is_dask_collection(x):
 
 def is_duck_dask_array(x):
     return is_duck_array(x) and is_dask_collection(x)
-
-
-dsk = DuckArrayModule("dask")
-dask_version = dsk.version
-dask_array_type = dsk.type
-
-sp = DuckArrayModule("sparse")
-sparse_array_type = sp.type
-sparse_version = sp.version
-
-cupy_array_type = DuckArrayModule("cupy").type