Skip to content
This repository has been archived by the owner on Sep 13, 2023. It is now read-only.

Commit

Permalink
Fix setup.py classifiers (#472)
Browse files Browse the repository at this point in the history
  • Loading branch information
aguschin authored Nov 19, 2022
1 parent a2427b5 commit e7753dd
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 12 deletions.
13 changes: 8 additions & 5 deletions .github/workflows/check-test-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ jobs:
strategy:
matrix:
os: [ubuntu-latest, windows-latest, macos-latest]
python: ["3.7", "3.8", "3.9", "3.10"]
python: ["3.8", "3.9", "3.10"]
exclude:
# no HDF5 support installed for tables
- os: windows-latest
Expand Down Expand Up @@ -96,7 +96,6 @@ jobs:
pip install --upgrade pip setuptools wheel
pip install pre-commit .[tests]
- run: pre-commit run pylint -a -v --show-diff-on-failure
if: matrix.python != '3.7'
- name: Start minikube
if: matrix.os == 'ubuntu-latest' && matrix.python == '3.9'
uses: medyagh/setup-minikube@master
Expand All @@ -113,14 +112,18 @@ jobs:
BITBUCKET_USERNAME: ${{ secrets.BITBUCKET_USERNAME }}
BITBUCKET_PASSWORD: ${{ secrets.BITBUCKET_PASSWORD }}
- name: Run Heroku tests
if: ${{ github.event_name == 'schedule' || contains(github.event.pull_request.title, 'heroku') }}
if: |
matrix.os == 'ubuntu-latest' &&
matrix.python == '3.8' &&
(
github.event_name == 'schedule' ||
contains(github.event.pull_request.title, 'heroku')
)
timeout-minutes: 40
run: pytest -k 'heroku'
env:
HEROKU_API_KEY: ${{ secrets.HEROKU_API_KEY }}
HEROKU_TEAM: iterative-sandbox
GITHUB_MATRIX_OS: ${{ matrix.os }}
GITHUB_MATRIX_PYTHON: ${{ matrix.python }}
- name: "Upload coverage to Codecov"
uses: codecov/codecov-action@v3
with:
Expand Down
3 changes: 1 addition & 2 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -129,10 +129,9 @@
classifiers=[
"Development Status :: 4 - Beta",
"Programming Language :: Python :: 3",
"Programming Language :: Python :: 3.6",
"Programming Language :: Python :: 3.7",
"Programming Language :: Python :: 3.8",
"Programming Language :: Python :: 3.9",
"Programming Language :: Python :: 3.10",
],
packages=find_packages(exclude=["tests*"]),
include_package_data=True,
Expand Down
6 changes: 1 addition & 5 deletions tests/contrib/test_heroku.py
Original file line number Diff line number Diff line change
Expand Up @@ -28,12 +28,11 @@
)
from mlem.core.errors import DeploymentError
from mlem.core.objects import DeployStatus, MlemModel
from tests.conftest import flaky, long, skip_matrix
from tests.conftest import flaky, long

heroku = pytest.mark.skipif(
HEROKU_CONFIG.API_KEY is None, reason="No HEROKU_API_KEY env provided"
)
heroku_matrix = skip_matrix("ubuntu-latest", "3.7")
HEROKU_TEST_APP_NAME_PREFIX = "mlem-test"
CLEAR_APPS = False
HEROKU_TEAM = os.environ.get("HEROKU_TEAM")
Expand Down Expand Up @@ -85,7 +84,6 @@ def test_heroku_api_request():

@heroku
@long
@heroku_matrix
def test_create_app(heroku_app_name, heroku_env, model):
name = heroku_app_name("create-app")
heroku_deploy = HerokuDeployment(
Expand All @@ -99,7 +97,6 @@ def test_create_app(heroku_app_name, heroku_env, model):


@long
@heroku_matrix
def test_build_heroku_docker(model: MlemModel, uses_docker_build):
image_meta = build_heroku_docker(model, "test_build", push=False)
client = DockerClient.from_env()
Expand Down Expand Up @@ -163,7 +160,6 @@ def is_not_crash(err, *args): # pylint: disable=unused-argument
@flaky(rerun_filter=is_not_crash, max_runs=1)
@heroku
@long
@heroku_matrix
def test_env_deploy_full(
tmp_path_factory,
model: MlemModel,
Expand Down

0 comments on commit e7753dd

Please sign in to comment.