Remove linting workflow in favor of pre-commit.ci (#120)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

README.md

@@ -1,10 +1,10 @@
 # POP-tools: Tools to support analysis of POP2-CESM model solutions with xarray
 
-| CI          | [![GitHub Workflow Status][github-ci-badge]][github-ci-link] [![GitHub Workflow Status][github-lint-badge]][github-lint-link] [![Code Coverage Status][codecov-badge]][codecov-link] |
-| :---------- | :----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------: |
-| **Docs**    |                                                                    [![Documentation Status][rtd-badge]][rtd-link]                                                                    |
-| **Package** |                                                         [![Conda][conda-badge]][conda-link] [![PyPI][pypi-badge]][pypi-link]                                                         |
-| **License** |                                                                        [![License][license-badge]][repo-link]                                                                        |
+| CI          | [![GitHub Workflow Status][github-ci-badge]][github-ci-link] [![Code Coverage Status][codecov-badge]][codecov-link] |
+| :---------- | :-----------------------------------------------------------------------------------------------------------------: |
+| **Docs**    |                                   [![Documentation Status][rtd-badge]][rtd-link]                                    |
+| **Package** |                        [![Conda][conda-badge]][conda-link] [![PyPI][pypi-badge]][pypi-link]                         |
+| **License** |                                       [![License][license-badge]][repo-link]                                        |
 
 ## Installation
 
@@ -23,9 +23,7 @@ conda install -c conda-forge pop-tools
 See [documentation](https://pop-tools.readthedocs.io) for more information.
 
 [github-ci-badge]: https://img.shields.io/github/workflow/status/NCAR/pop-tools/CI?label=CI&logo=github&style=for-the-badge
-[github-lint-badge]: https://img.shields.io/github/workflow/status/NCAR/pop-tools/linting?label=linting&logo=github&style=for-the-badge
 [github-ci-link]: https://github.com/NCAR/pop-tools/actions?query=workflow%3ACI
-[github-lint-link]: https://github.com/NCAR/pop-tools/actions?query=workflow%3Alinting
 [codecov-badge]: https://img.shields.io/codecov/c/github/NCAR/pop-tools.svg?logo=codecov&style=for-the-badge
 [codecov-link]: https://codecov.io/gh/NCAR/pop-tools
 [rtd-badge]: https://img.shields.io/readthedocs/pop-tools/latest.svg?style=for-the-badge

docs/source/conf.py

@@ -1,5 +1,4 @@
 #!/usr/bin/env python
-# -*- coding: utf-8 -*-
 
 import datetime
 import inspect
@@ -109,9 +108,9 @@
 
 # General information about the project.
 current_year = datetime.datetime.now().year
-project = u'pop-tools'
-copyright = u'2018-{}, University Corporation for Atmospheric Research'.format(current_year)
-author = u'pop-tools developers'
+project = 'pop-tools'
+copyright = f'2018-{current_year}, University Corporation for Atmospheric Research'
+author = 'pop-tools developers'
 
 # The version info for the project you're documenting, acts as replacement
 # for |version| and |release|, also used in various other places throughout
@@ -221,14 +220,14 @@
 # Grouping the document tree into LaTeX files. List of tuples
 # (source start file, target name, title, author, documentclass
 # [howto, manual, or own class]).
-latex_documents = [(master_doc, 'pop-tools.tex', u'POP-tools Documentation', author, 'manual')]
+latex_documents = [(master_doc, 'pop-tools.tex', 'POP-tools Documentation', author, 'manual')]
 
 
 # -- Options for manual page output ------------------------------------
 
 # One entry per manual page. List of tuples
 # (source start file, name, description, authors, manual section).
-man_pages = [(master_doc, 'pop-tools', u'POP-tools Documentation', [author], 1)]
+man_pages = [(master_doc, 'pop-tools', 'POP-tools Documentation', [author], 1)]
 
 
 # -- Options for Texinfo output ----------------------------------------
@@ -240,7 +239,7 @@
     (
         master_doc,
         'POP-tools',
-        u'POP-tools Documentation',
+        'POP-tools Documentation',
         author,
         'POP-tools',
         'One line description of project.',

setup.cfg

@@ -11,7 +11,7 @@ test = pytest
 
 [tool:pytest]
 console_output_style = count
-addopts = -n auto --cov=./ --cov-report=xml --verbose
+addopts = --cov=./ --cov-report=xml --verbose
 
 [isort]
 known_first_party=pop_tools

tests/test_fill.py

@@ -91,7 +91,7 @@ def test_lateral_fill_3D():
     field = ds.KMT.copy() * 1.0
     field = field.where(ds.KMT > 0)
     field.values[20:40, 80:] = np.nan
-    da_in = xr.DataArray(np.ones((3)), dims=('z_t')) * field
+    da_in = xr.DataArray(np.ones(3), dims=('z_t')) * field
     attrs = {'long_name': 'test field', 'units': 'none'}
     da_in.attrs = attrs
 
@@ -114,9 +114,7 @@ def test_lateral_fill_4D():
     field = field.where(ds.KMT > 0)
     field.values[20:40, 80:] = np.nan
 
-    da_in = (
-        xr.DataArray(np.ones((3)), dims=('time')) * xr.DataArray(np.ones((5)), dims=('z_t')) * field
-    )
+    da_in = xr.DataArray(np.ones(3), dims=('time')) * xr.DataArray(np.ones(5), dims=('z_t')) * field
 
     attrs = {'long_name': 'test field', 'units': 'none'}
     da_in.attrs = attrs
@@ -140,8 +138,8 @@ def test_lateral_fill_4D_3Dmask():
     field.values[20:40, 80:] = np.nan
 
     da_in = (
-        xr.DataArray(np.ones((3)), dims=('time'))
-        * xr.DataArray(np.ones((len(ds.z_t))), dims=('z_t'))
+        xr.DataArray(np.ones(3), dims=('time'))
+        * xr.DataArray(np.ones(len(ds.z_t)), dims=('z_t'))
         * field
     )