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

[WIP] Fix recording output format #214

Closed
wants to merge 3 commits into from

Conversation

mithro
Copy link
Member

@mithro mithro commented May 4, 2015

No description provided.

mithro added 2 commits May 4, 2015 22:43
${PWD} is used in python-api/Makefile to set the PYTHONPATH. When using -C the
value of PWD remains the top level directory rather than the python-api
directory leading to import-errors when running pylint.
make PYTHONVERSION=2.7 -Cpython-api style unittests integration
make PYTHONVERSION=3.4 -Cpython-api style unittests integration
cd python-api; make PYTHONVERSION=2.7 style unittests integration
cd python-api; make PYTHONVERSION=3.4 style unittests integration

Choose a reason for hiding this comment

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

why exactly are you changing this?
ah, just read the commit message

Copy link
Member Author

Choose a reason for hiding this comment

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

Oops that should be in a different pull request. See the comment on the commit bd70921

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling b446658 on mithro:recording-fix into 9ad792a on timvideos:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 99.91% when pulling b446658 on mithro:recording-fix into 9ad792a on timvideos:master.

@coveralls
Copy link

coveralls commented Aug 4, 2016

Coverage Status

Coverage decreased (-99.9%) to 0.0% when pulling 4738903 on mithro:recording-fix into 9ad792a on timvideos:master.

@mithro mithro closed this Mar 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants