Skip to content

[EXP] pytest3 stuff - #63

Closed
pllim wants to merge 3 commits into
astrofrog:remove-bundled-pytestfrom
pllim:astrofrog-pytest3-1
Closed

pllim wants to merge 3 commits into
astrofrog:remove-bundled-pytestfrom
pllim:astrofrog-pytest3-1

Conversation

@pllim

@pllim pllim commented Jan 18, 2017 •

Copy link
Copy Markdown

Some changes from mhvk#23 that did not make it into astropy#5688. You don't have to merge this. Just take what you need to make it work. This is a PR against astropy#5694.

@pllim pllim changed the title pytest3 stuff [EXP] pytest3 stuff Jan 18, 2017
Comment thread .travis.yml
matrix:
- PYTHON_VERSION=2.7 SETUP_CMD='egg_info'
- PYTHON_VERSION=3.3 SETUP_CMD='egg_info'
- PYTHON_VERSION=3.3 SETUP_CMD='egg_info' PIP_DEPENDENCIES='pytest>=3'

@pllim pllim Jan 18, 2017 •

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note: This is needed or Travis is going to fall back to pytest 2.7 for Python 3.3. See this comment in #5688.

Comment thread setup.cfg

[tool:pytest]
minversion = 2.3.3
minversion = 3

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not really sure what this one does but at least it should be consistent?

Comment thread .travis.yml
env:
global:
# Set defaults to avoid repeating in most cases
- ASTROPY_USE_SYSTEM_PYTEST=1

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We might not need this anymore? Not sure.

@astrofrog
astrofrog force-pushed the remove-bundled-pytest branch 3 times, most recently from 6d91cd3 to cbdd7c4 Compare January 20, 2017 12:52
@pllim pllim closed this Jan 20, 2017
@pllim
pllim deleted the astrofrog-pytest3-1 branch January 20, 2017 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants