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

MRG: CircleCI 2.0 #350

Merged
merged 10 commits into from May 4, 2018
Merged

MRG: CircleCI 2.0 #350

merged 10 commits into from May 4, 2018

Conversation

larsoner
Copy link
Member

Closes #349.

@larsoner larsoner force-pushed the circle branch 5 times, most recently from d4f60aa to b80843f Compare April 27, 2018 18:39
@larsoner larsoner force-pushed the circle branch 3 times, most recently from 593b4a2 to af8a04d Compare April 27, 2018 18:50
@larsoner larsoner changed the title FIX: CircleCI 2.0 MRG: CircleCI 2.0 Apr 27, 2018
@larsoner
Copy link
Member Author

@drammock ready for review/merge from my end

@drammock
Copy link
Member

I can review this on monday when I'm back in the office

@drammock drammock self-requested a review April 27, 2018 23:35
@drammock drammock assigned drammock and unassigned drammock Apr 27, 2018
@drammock
Copy link
Member

drammock commented May 3, 2018

(three) [~/B/expyfun] $ git chb testcircle eric/circle 
Branch testcircle set up to track remote branch circle from eric.
Switched to a new branch 'testcircle'
(three) [~/B/expyfun] $ make test
rm -rf build
find . -name "*.pyc" | xargs rm -f
find . -name "*.so" | xargs rm -f
find . -name "*.pyd" | xargs rm -f
rm -f tags
find . -name "__pycache__" | xargs rm -rf
rm -f .coverage
pytest expyfun
usage: pytest [options] [file_or_dir] [file_or_dir] [...]
pytest: error: unrecognized arguments: --cov=expyfun --cov-report=
  inifile: /home/drmccloy/Builds/expyfun/setup.cfg
  rootdir: /home/drmccloy/Builds/expyfun
Makefile:44: recipe for target 'nosetests' failed
make: *** [nosetests] Error 2

@larsoner
Copy link
Member Author

larsoner commented May 3, 2018

install pytest-cov

@drammock
Copy link
Member

drammock commented May 3, 2018

is my version of pytest too old? (maybe we should check for that?)

@larsoner
Copy link
Member Author

larsoner commented May 3, 2018

No you need both pytest and pytest-cov packages

@drammock
Copy link
Member

drammock commented May 3, 2018

test pass in my python3 env, but in my (rarely-used) py2 environment I'm getting an enormous traceback culminating in RuntimeError: maximum recursion depth exceeded. Should I ignore it or try to track it down?

@drammock
Copy link
Member

drammock commented May 3, 2018

found problem. Old pip-installed version of py was overriding newer conda-installed version of py. sigh

@larsoner
Copy link
Member Author

larsoner commented May 3, 2018

https://xkcd.com/1987/

@drammock
Copy link
Member

drammock commented May 3, 2018

today is not my day:

(three) [~] $ conda install argcomplete
Solving environment: - Segmentation fault (core dumped)

@larsoner
Copy link
Member Author

larsoner commented May 3, 2018

Oh wow, never seen that one before.

Copy link
Member

@drammock drammock left a comment

Choose a reason for hiding this comment

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

other than one comment LGTM

ex = want_version
if want_version == 'cae6bc3':
ex = (ex, '.dev0+c')
assert got_version in ex, got_fname
Copy link
Member

Choose a reason for hiding this comment

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

this line I don't get. why is got_fname here?

Copy link
Member

Choose a reason for hiding this comment

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

(i.e., why would the got_version ever equal the file path, and why would we want to allow that?)

Copy link
Member Author

Choose a reason for hiding this comment

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

In assert statements what follows the comma is a message (it is not creating a tuple), e.g.:

>>> assert 'dan' in 'cool', 'he is not'
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AssertionError: he is not

In this case it's useful to have the filename printed, especially when using pytest --pdb.

Copy link
Member

Choose a reason for hiding this comment

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

I've been pwned

@drammock drammock merged commit f6e849a into LABSN:master May 4, 2018
@drammock
Copy link
Member

drammock commented May 4, 2018

thanks @larsoner

@larsoner larsoner deleted the circle branch May 4, 2018 01:43
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.

None yet

2 participants