Skip to content

Remove scripts except pull-static #1519

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

Merged
merged 2 commits into from
Feb 3, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
3 changes: 2 additions & 1 deletion .github/pull_request_template.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,8 @@

**PR Checklist:**

- [ ] Pre-commit hooks and tests pass (run `scripts/test`)
- [ ] Pre-commit hooks pass (run `pre-commit run --all-files`)
- [ ] Tests pass (run `pytest`)
- [ ] Documentation has been updated to reflect changes, if applicable
- [ ] This PR maintains or improves overall codebase code coverage.
- [ ] Changes are added to the [CHANGELOG](https://github.com/stac-utils/pystac/blob/main/CHANGELOG.md). See [the docs](https://pystac.readthedocs.io/en/latest/contributing.html#changelog) for information about adding to the changelog.
2 changes: 1 addition & 1 deletion .github/workflows/continuous-integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ jobs:
run: uv run pytest tests
- name: Test
if: runner.os != 'Windows'
run: uv run scripts/test
run: uv run pytest tests --block-network --record-mode=none

coverage:
name: coverage
Expand Down
4 changes: 2 additions & 2 deletions docs/contributing.rst
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ To run the tests and generate the coverage report:

.. code-block:: bash

$ pytest -v -s --block-network --cov pystac --cov-report term-missing
$ pytest -v -s --cov pystac --cov-report term-missing

To view the coverage report, you can run
`coverage report` (to view the report in the terminal) or `coverage html` (to generate
Expand Down Expand Up @@ -100,7 +100,7 @@ and report any improvements or regressions.

.. code-block:: bash

scripts/bench
asv continuous --split -e --interleave-rounds --factor 1.25 main HEAD

The benchmark suite takes a while to run, and will report any significant
changes to standard output. For example, here's a benchmark comparison between
Expand Down
1 change: 1 addition & 0 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,7 @@ lint.select = ["E", "F", "I"]

[tool.pytest.ini_options]
filterwarnings = ["error"]
addopts = "--block-network --record-mode=none"

[tool.setuptools.packages.find]
include = ["pystac*"]
Expand Down
11 changes: 0 additions & 11 deletions scripts/bench

This file was deleted.

2 changes: 1 addition & 1 deletion scripts/pull-static
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

set -e

VERSION="1.5.0" #v1.5.0-beta.2 should work or no??
VERSION="1.5.0"
SRC="https://cdn.jsdelivr.net/npm/@radiantearth/stac-fields@$VERSION/fields-normalized.json"
HERE=$(dirname "$0")
DEST=$(dirname "$HERE")/pystac/static/fields-normalized.json
Expand Down
18 changes: 0 additions & 18 deletions scripts/test

This file was deleted.

Loading