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

test worker instead of slave #44

Merged
merged 2 commits into from Jun 27, 2016
Merged

test worker instead of slave #44

merged 2 commits into from Jun 27, 2016

Conversation

tardyp
Copy link
Member

@tardyp tardyp commented Jun 27, 2016

No description provided.

@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @tomprince, @djmitche and @rutsky to be potential reviewers

@rutsky
Copy link
Member

rutsky commented Jun 27, 2016

Linter yells:

./status.py:3:1: E402 module level import not at top of file

from buildbot.status import html

^

./status.py:4:1: E402 module level import not at top of file

from buildbot.status import words

^

./status.py:5:1: E402 module level import not at top of file

from buildbot.status.web.auth import BasicAuth

^

./status.py:6:1: E402 module level import not at top of file

from buildbot.status.web.authz import Authz

^

4       E402 module level import not at top of file

4

@tomprince
Copy link
Member

The lint failures are due to flake8 2.6 (see #45).

@tardyp tardyp merged commit bee0d7d into buildbot:master Jun 27, 2016
@tomprince
Copy link
Member

@tardyp: Thanks for fixing this up.

@tardyp
Copy link
Member Author

tardyp commented Jun 27, 2016

@tomprince good teamwork

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