diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index 10991facb3..a0d41f9841 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -5,5 +5,5 @@ TODO: * [ ] Add docstrings and API docs for any new/modified user-facing classes and functions * [ ] New/modified features documented in docs/tutorial.rst * [ ] Changes documented in docs/release.rst -* [ ] AppVeyor and Travis CI passes -* [ ] Test coverage is 100% (Coveralls passes) +* [ ] GitHub Actions have all passed +* [ ] Test coverage is 100% (Codecov passes) diff --git a/appveyor.yml b/appveyor.yml deleted file mode 100644 index 271abeb233..0000000000 --- a/appveyor.yml +++ /dev/null @@ -1,51 +0,0 @@ -branches: - only: - - master - -# the VS C++ compiler path, doesn't seem to exist in the PATH environment variable of -# the Visual Studio 2017 build VM, due to which the pyosreplace package fails to build -image: Visual Studio 2015 - -environment: - - global: - # SDK v7.0 MSVC Express 2008's SetEnv.cmd script will fail if the - # /E:ON and /V:ON options are not enabled in the batch script intepreter - # See: http://stackoverflow.com/a/13751649/163740 - CMD_IN_ENV: "cmd /E:ON /V:ON /C .\\build.cmd" - EMULATOR_LOC: C:\\Program Files (x86)\\Microsoft SDKs\\Azure\\Storage Emulator\\AzureStorageEmulator.exe - - matrix: - - PYTHON: "C:\\Python36-x64" - PYTHON_VERSION: "3.6" - ZARR_TEST_ABS: 0 - - - PYTHON: "C:\\Python37-x64" - PYTHON_VERSION: "3.7" - ZARR_TEST_ABS: 0 - - - PYTHON: "C:\\Python38-x64" - PYTHON_VERSION: "3.8" - ZARR_TEST_ABS: 1 - -install: - - "SET PATH=%PYTHON%;%PYTHON%\\Scripts;%PATH%" - - "%CMD_IN_ENV% python -m pip install -U pip setuptools wheel" - - "%CMD_IN_ENV% python -m pip install -r requirements_dev_numpy.txt -r requirements_dev_minimal.txt -r requirements_dev_optional.txt" - - "%CMD_IN_ENV% python setup.py install" - - "%CMD_IN_ENV% python -m pip freeze" - -build: off - -before_test: - - '"%EMULATOR_LOC%" start' - - '"%EMULATOR_LOC%" status' - -test_script: - - "%CMD_IN_ENV% python -m pytest -v --pyargs zarr" - -after_test: - - '"%EMULATOR_LOC%" stop' - -matrix: - fast_finish: true diff --git a/docs/contributing.rst b/docs/contributing.rst index b129c983e3..06c4abbba0 100644 --- a/docs/contributing.rst +++ b/docs/contributing.rst @@ -169,13 +169,11 @@ server locally and set the environment variable ``ZARR_TEST_MONGO=1``. To run the Redis storage tests, run a Redis server locally on port 6379 and set the environment variable ``ZARR_TEST_REDIS=1``. -All tests are automatically run via Travis (Linux) and AppVeyor -(Windows) continuous integration services for every pull -request. Tests must pass under both Travis and Appveyor before code -can be accepted. Test coverage is also collected automatically via the -Coveralls service, and total coverage over all builds must be 100% -(although individual builds may be lower due to Python 2/3 or other -differences). +All tests are automatically run via GitHub Actions for every pull +request and must pass before code can be accepted. Test coverage is +also collected automatically via the Codecov service, and total +coverage over all builds must be 100% (although individual builds +may be lower due to Python 2/3 or other differences). Code standards ~~~~~~~~~~~~~~ @@ -196,8 +194,8 @@ and produce a coverage report. This should be 100% before code can be accepted i main code base. When submitting a pull request, coverage will also be collected across all supported -Python versions via the Coveralls service, and will be reported back within the pull -request. Coveralls coverage must also be 100% before code can be accepted. +Python versions via the Codecov service, and will be reported back within the pull +request. Codecov coverage must also be 100% before code can be accepted. Documentation ~~~~~~~~~~~~~ @@ -239,8 +237,8 @@ Pull requests submitted by an external contributor should be reviewed and approv one core developers before being merged. Ideally, pull requests submitted by a core developer should be reviewed and approved by at least one other core developers before being merged. -Pull requests should not be merged until all CI checks have passed (Travis, AppVeyor, -Coveralls) against code that has had the latest master merged in. +Pull requests should not be merged until all CI checks have passed (GitHub Actions +Codecov) against code that has had the latest master merged in. Compatibility and versioning policies ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ diff --git a/requirements_dev_optional.txt b/requirements_dev_optional.txt index 697758dac7..cbc9aca562 100644 --- a/requirements_dev_optional.txt +++ b/requirements_dev_optional.txt @@ -5,7 +5,7 @@ lmdb==0.97; sys_platform != 'win32' ipytree==0.1.3 # optional library requirements for services # don't let pyup change pinning for azure-storage-blob, need to pin to older -# version to get compatibility with azure storage emulator on appveyor +# version to get compatibility with azure storage emulator on appveyor (FIXME) azure-storage-blob==2.0.1 # pyup: ignore redis==3.3.8 pymongo==3.9.0