-
Notifications
You must be signed in to change notification settings - Fork 6.5k
Use new nox #1765
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
Use new nox #1765
Conversation
@engelke Would you be able to review this? |
I'll check it out. I'm also looking at specifying a specific container in
the config so that changing the container will be visible as Github change,
too.
…On Tue, Oct 16, 2018 at 9:16 AM Bu Sun Kim ***@***.***> wrote:
@engelke <https://github.com/engelke> Would you be able to review this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1765 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAE_Ny1R9pRJ2ziLAtgi21tkfvSKQfouks5ulgZvgaJpZM4XdVmB>
.
--
Charles Engelke
GCP AppDev DPE
|
Seems to be a problem with nox under Python 2.7, which Travis is using for lint. Here's the error message: Traceback (most recent call last): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any remaining configuration issues need to be addressed after merge is in place.
Changes were moved to PR 1770 in order to add broader changes needed to pass Travis check. |
Closes #1736.