Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

release v1.3.0 #611

Merged
merged 31 commits into from
Sep 16, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
5825e51
Update viz doc to clarify its current status (partial functionality -…
mfisher87 Aug 14, 2024
4648551
Avoid running tests that require creds on PRs (#563)
mfisher87 Aug 16, 2024
b0bb832
Skip linting and formatting Jupyter Notebooks for now (#567)
mfisher87 Aug 16, 2024
8b57fca
Keep GitHub Actions up to date with GitHub's Dependabot (#555)
cclauss Aug 16, 2024
a259f9a
Bump the github-actions group with 3 updates (#568)
dependabot[bot] Aug 17, 2024
8ef252c
Add codespell spelling/typo linter and fix errors (#556)
cclauss Aug 18, 2024
118d49a
Validate pyproject.toml (#557)
cclauss Aug 18, 2024
687766d
Enable currently-ignored linting rules E721, E722, F403 (#573)
cclauss Aug 19, 2024
3c06edb
Add ruff rules C4 to check comprehensions (#570)
cclauss Aug 19, 2024
fca67b9
Bugfix: catch multiple types of exceptions (#560)
mfisher87 Aug 20, 2024
6c6366a
Add quickstart to README (#549)
mfisher87 Aug 21, 2024
cd275e7
Add ruff rules SIM to simplify Python code (#572)
cclauss Aug 23, 2024
bc39b7c
docs: add cclauss as a contributor for maintenance and review (#583)
allcontributors[bot] Aug 23, 2024
a762ad8
docs: add jrenrut as a contributor for bug, and code (#586)
allcontributors[bot] Aug 23, 2024
e491c84
docs: add mfisher87 as a contributor for bug, code, and 4 more (#585)
mfisher87 Aug 23, 2024
2790edd
Tweak format and wording of contributor doc (#566)
mfisher87 Aug 26, 2024
ee43461
More ruff & pre-commit rules (#546)
mfisher87 Aug 27, 2024
50f9351
Fix minor typo (#589)
mfisher87 Aug 27, 2024
a52bc37
Clarify some comments / docstrings (#592)
mfisher87 Aug 27, 2024
bd09d73
Migrate unit and integration tests from TravisCI to GitHub Actions (#…
mfisher87 Aug 29, 2024
572e48c
Extract constant URLs to own module (#591)
mfisher87 Aug 30, 2024
6d6a085
Reinstate test_download_granules_without_subsetting (#581)
weiji14 Sep 3, 2024
418c9e7
Bump sangonzal/repository-traffic-action from 0.1.5 to 1 in the githu…
dependabot[bot] Sep 5, 2024
dbaff0e
[pre-commit.ci] pre-commit autoupdate (#597)
pre-commit-ci[bot] Sep 5, 2024
054fbe4
bump sangonzal/repository-traffic-action version to latest working on…
JessicaS11 Sep 9, 2024
1cf6504
bump sangonzal/repository-traffic-action version to latest working on…
JessicaS11 Sep 9, 2024
2566d96
[docs] traffic updates May-Aug 2024 (#606)
JessicaS11 Sep 12, 2024
bea9fe0
Drop support for Python 3.7 and 3.8 (#608)
mfisher87 Sep 13, 2024
a98409f
Add v1.x deprecation warning (#603)
mfisher87 Sep 16, 2024
1439669
Replace integration test review trigger with manual trigger (#595)
mfisher87 Sep 16, 2024
2495978
release v1.3.0 (#607)
JessicaS11 Sep 16, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
35 changes: 35 additions & 0 deletions .all-contributorsrc
Original file line number Diff line number Diff line change
Expand Up @@ -470,6 +470,41 @@
"contributions": [
"ideas"
]
},
{
"login": "cclauss",
"name": "Christian Clauss",
"avatar_url": "https://avatars.githubusercontent.com/u/3709715?v=4",
"profile": "https://github.com/cclauss",
"contributions": [
"maintenance",
"review",
"tool"
]
},
{
"login": "jrenrut",
"name": "Jeremy Turner",
"avatar_url": "https://avatars.githubusercontent.com/u/42989238?v=4",
"profile": "https://github.com/jrenrut",
"contributions": [
"bug",
"code"
]
},
{
"login": "mfisher87",
"name": "Matt Fisher",
"avatar_url": "https://avatars.githubusercontent.com/u/3608264?v=4",
"profile": "https://mfisher87.github.io/",
"contributions": [
"bug",
"code",
"doc",
"maintenance",
"test",
"infra"
]
}
],
"contributorsPerLine": 7,
Expand Down
19 changes: 19 additions & 0 deletions .github/actions/install-icepyx/action.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
name: 'Install icepyx'
description: 'Install icepyx and dev dependencies'

inputs:
python-version:
required: true

runs:
using: "composite"
steps:
- uses: "actions/setup-python@v5"
with:
python-version: "${{ inputs.python-version }}"

- name: "Install package and test dependencies"
shell: "bash"
run: |
python -m pip install .
python -m pip install -r requirements-dev.txt
13 changes: 13 additions & 0 deletions .github/dependabot.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
# Keep GitHub Actions up to date with GitHub's Dependabot...
# https://docs.github.com/en/code-security/dependabot/working-with-dependabot/keeping-your-actions-up-to-date-with-dependabot
# https://docs.github.com/en/code-security/dependabot/dependabot-version-updates/configuration-options-for-the-dependabot.yml-file#package-ecosystem
version: 2
updates:
- package-ecosystem: github-actions
directory: /
groups:
github-actions:
patterns:
- "*" # Group all Actions updates into a single larger pull request
schedule:
interval: monthly
2 changes: 1 addition & 1 deletion .github/workflows/get_pypi_stats.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ jobs:
# Steps represent a sequence of tasks that will be executed as part of the job
steps:
# Checks-out your repository under $GITHUB_WORKSPACE, so your job can access it
- uses: actions/checkout@v3
- uses: actions/checkout@v4
with:
ref: "traffic"

Expand Down
49 changes: 49 additions & 0 deletions .github/workflows/integration_test.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,49 @@
name: "Integration test"
# NOTE: We're just running the tests that require earthdata login here; we
# don't distinguish between unit and integration tests yet.

on:
push:
branches:
- "main" # Releases
- "development" # Feature PR merges
pull_request:
branches:
- "main" # Release PRs
workflow_dispatch:
inputs:
ref:
description: "The ref to test"
type: "string"


jobs:
test:
name: "Integration test"
# Do not run on PRs from forks:
if: "${{ !github.event.pull_request.head.repo.fork }}"
runs-on: "ubuntu-latest"

steps:
- uses: "actions/checkout@v4"
with:
fetch-depth: 0
ref: "${{ inputs.ref }}"

- uses: "./.github/actions/install-icepyx"
with:
python-version: "3.12"

- name: "Run tests"
env:
EARTHDATA_PASSWORD: "${{ secrets.EARTHDATA_PASSWORD }}"
NSIDC_LOGIN: "${{ secrets.EARTHDATA_PASSWORD }}"
run: |
pytest icepyx/ --verbose --cov app \
icepyx/tests/test_behind_NSIDC_API_login.py \
icepyx/tests/test_auth.py

- name: "Upload coverage report"
uses: "codecov/[email protected]"
with:
token: "${{ secrets.CODECOV_TOKEN }}"
4 changes: 2 additions & 2 deletions .github/workflows/publish_to_pypi.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,13 +22,13 @@ jobs:

steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
# fetch all history so that setuptools-scm works
fetch-depth: 0

- name: Set up Python
uses: actions/setup-python@v4
uses: actions/setup-python@v5
with:
python-version: 3.9

Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/traffic_action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,13 +17,13 @@ jobs:
# Steps represent a sequence of tasks that will be executed as part of the job
steps:
# Checks-out your repository under $GITHUB_WORKSPACE, so your job can access it
- uses: actions/checkout@v3
- uses: actions/checkout@v4
with:
ref: "traffic"

# Calculates traffic and clones and stores in CSV file
- name: GitHub traffic
uses: sangonzal/repository-traffic-action@v0.1.4
uses: sangonzal/repository-traffic-action@v.0.1.6
env:
TRAFFIC_ACTION_TOKEN: ${{ secrets.TRAFFIC_ACTION_TOKEN }}

Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/uml_action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Check out repository
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
ref: ${{ github.event.pull_request.head.ref }}
- name: set up environment
Expand Down
40 changes: 40 additions & 0 deletions .github/workflows/unit_test.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,40 @@
name: "Unit test"
# NOTE: We're just skipping the tests requiring earthdata login here; we don't
# distinguish yet between unit and integration tests.

on:
pull_request:
push:
branches:
- "main"
- "development"


jobs:
test:
name: "Unit test (Python ${{ matrix.python-version }})"
runs-on: "ubuntu-latest"
strategy:
fail-fast: false
matrix:
python-version: ["3.9", "3.12"] #NOTE: min and max Python versions supported by icepyx

steps:
- uses: "actions/checkout@v4"
with:
fetch-depth: 0

- uses: "./.github/actions/install-icepyx"
with:
python-version: "${{ matrix.python-version }}"

- name: "Run tests"
run: |
pytest icepyx/ --verbose --cov app \
--ignore=icepyx/tests/test_behind_NSIDC_API_login.py \
--ignore=icepyx/tests/test_auth.py

- name: "Upload coverage report"
uses: "codecov/[email protected]"
with:
token: "${{ secrets.CODECOV_TOKEN }}"
22 changes: 21 additions & 1 deletion .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,17 +6,37 @@ repos:
- id: check-yaml
- id: check-added-large-files
args: ["--maxkb=5000"]
- id: check-case-conflict
- id: check-merge-conflict
- id: check-symlinks
- id: debug-statements
- id: name-tests-test
args: ["--pytest-test-first"]
- id: end-of-file-fixer
- id: mixed-line-ending
- id: trailing-whitespace
args: [--markdown-linebreak-ext=md]
- id: requirements-txt-fixer

- repo: https://github.com/codespell-project/codespell
rev: v2.3.0
hooks:
- id: codespell
additional_dependencies:
- tomli

- repo: https://github.com/astral-sh/ruff-pre-commit
rev: v0.5.7
rev: v0.6.3
hooks:
- id: ruff
args: ["--fix", "--show-fixes"]
- id: ruff-format

- repo: https://github.com/abravalheri/validate-pyproject
rev: v0.19
hooks:
- id: validate-pyproject

ci:
autoupdate_schedule: monthly

Expand Down
44 changes: 0 additions & 44 deletions .travis.yml

This file was deleted.

2 changes: 1 addition & 1 deletion ATTRIBUTION.rst
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ to identify core issues and propose solutions to challenges around software cred
For software, current best practices (`e.g. <https://arxiv.org/pdf/2012.13117.pdf>`_) emphasize the importance of having a document
such as this one to describe an individual community's policies for credit, authorship, and attribution.
This document is an effort to describe icepyx's policies, with an awareness that they may change
to accomodate community growth, best practices, and feedback.
to accommodate community growth, best practices, and feedback.

We do not attempt to identify contribution levels through the number of commits made to the repository (e.g. ``git shortlog -sne``)
or active engagement on GitHub (e.g. through issues, discussions, and pull requests) and Discourse.
Expand Down
Loading