Skip to content

Commit

Permalink
Update minimum supported version to Python 3.5
Browse files Browse the repository at this point in the history
  • Loading branch information
mperrin committed Sep 20, 2018
1 parent c8de6e0 commit 5c08e37
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 7 deletions.
6 changes: 3 additions & 3 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,10 +49,10 @@ matrix:
- python: 3.6
env: SETUP_CMD='test' ASTROPY_VERSION=development

# Try older numpy versions
- python: 3.4
# Try minimum supported versions
- python: 3.5
env: SETUP_CMD='test' NUMPY_VERSION=1.12
- python: 3.4
- python: 3.5
env: SETUP_CMD='test' NUMPY_VERSION=1.11

# Try released POPPY
Expand Down
16 changes: 12 additions & 4 deletions setup.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
#!/usr/bin/env python
# Based on astropy affiliated package template's setup.py
# Licensed under a 3-clause BSD style license - see LICENSE.rst
# --based on setup.py from astropy--
from __future__ import print_function

import glob
Expand All @@ -9,6 +9,15 @@
import imp
import ast

__packagename__ = 'webbpsf'
__minimum_python_version__ = "3.5"

# Enforce Python version check - this is the same check as in __init__.py but
# this one has to happen before importing ah_bootstrap.
if sys.version_info < tuple((int(val) for val in __minimum_python_version__.split('.'))):
sys.stderr.write("ERROR: {} requires Python {} or later\n".format(__packagename__, __minimum_python_version__))
sys.exit(1)

try:
import numpy
except ImportError:
Expand Down Expand Up @@ -109,16 +118,15 @@
version=VERSION,
description=DESCRIPTION,
scripts=scripts,
python_requires='>=' + __minimum_python_version__,
install_requires=[
'numpy>=1.10.0',
'matplotlib>=1.5.0',
'scipy>=0.16.0',
'poppy>=0.7.0',
'astropy>=1.2.0',
'astropy>=1.3.0',
'jwxml>=0.3.0',
'pysiaf>=0.1.8', 'six',
'pytest' # unlisted requirement for pysiaf - see https://github.com/spacetelescope/pysiaf/issues/16
# Remove this requirement once that issue is addressed.
],
provides=[PACKAGENAME],
author=AUTHOR,
Expand Down

0 comments on commit 5c08e37

Please sign in to comment.