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.

Sign up for GitHub

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

Jump to bottom

F26 dev fix pylint #1027

Merged
merged 1 commit into from Apr 19, 2017
Merged

F26 dev fix pylint #1027

merged 1 commit into from Apr 19, 2017

Conversation

jkonecny12
Copy link
Member

Fix nosetest and pylint errors on F26 and master.

@jkonecny12 jkonecny12 added the f26 label Apr 18, 2017
M4rtinK
M4rtinK approved these changes Apr 18, 2017
View reviewed changes
Copy link
Contributor

@M4rtinK M4rtinK left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@jkonecny12 jkonecny12 mentioned this pull request Apr 19, 2017
Merged
@jkonecny12
Copy link
Member Author

Fix nosetest commit was removed. It is already fixed by #1021 .

@jkonecny12
Copy link
Member Author

jenkins, test this please

@jkonecny12 jkonecny12 merged commit 4c2a9f7 into rhinstaller:f26-devel Apr 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@M4rtinK M4rtinK M4rtinK approved these changes

@rvykydal rvykydal Awaiting requested review from rvykydal

Assignees
No one assigned
Labels
f26
Milestone
No milestone
2 participants
@jkonecny12 @M4rtinK