Skip to content

Conversation

@theacodes
Copy link

Note: has #32 as a diffbase (until it gets merged).

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 3, 2016
@theacodes theacodes force-pushed the pytest-refactor-nox branch 7 times, most recently from 378e326 to 6e329ad Compare March 3, 2016 19:34
@theacodes theacodes changed the title Pytest refactor nox Use nox as the top-level test runner, use gcprepotools for req sessions Mar 3, 2016
# Copyright 2015 Google Inc.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm confused why this is part of this PR.

Copy link
Author

Choose a reason for hiding this comment

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

It had a diffbase, hold on, let me update now that the other PR is merged.

@theacodes theacodes force-pushed the pytest-refactor-nox branch from 6e329ad to e0ebd5b Compare March 3, 2016 20:54
# Unit test / coverage reports
htmlcov/
.tox/
.nox/
Copy link
Contributor

Choose a reason for hiding this comment

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

super nit - can get rid of .tox here

Copy link
Author

Choose a reason for hiding this comment

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

Unfortunately we can't, because the individual steps still use tox and will create .tox folders. ;)

@waprin
Copy link
Contributor

waprin commented Mar 3, 2016

lgtm

theacodes pushed a commit that referenced this pull request Mar 3, 2016
Use nox as the top-level test runner, use gcprepotools for req sessions
@theacodes theacodes merged commit 76807fa into pytest-refactor Mar 3, 2016
@theacodes theacodes deleted the pytest-refactor-nox branch March 3, 2016 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants