Skip to content
This repository has been archived by the owner on Sep 23, 2024. It is now read-only.

Remove unused variables #144

Merged
merged 1 commit into from
May 22, 2018
Merged

Remove unused variables #144

merged 1 commit into from
May 22, 2018

Conversation

major
Copy link
Contributor

@major major commented May 21, 2018

This PR works towards the goals of #3.

Signed-off-by: Major Hayden [email protected]

@major
Remove unused variables
b97c22a 
Signed-off-by: Major Hayden <[email protected]>
@major major added the enhancement New feature or request label May 21, 2018
@major major self-assigned this May 21, 2018
@coveralls
Copy link

Pull Request Test Coverage Report for Build 468

  • 3 of 13 (23.08%) changed or added relevant lines in 4 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 46.658%
Changes Missing Coverage Covered Lines Changed/Added Lines %
skt/runner.py 3 4 75.0%
skt/executable.py 0 1 0.0%
skt/reporter.py 0 1 0.0%
skt/init.py 0 7 0.0%
Totals Coverage Status
Change from base Build 463: 0.0%
Covered Lines: 684
Relevant Lines: 1466
💛 - Coveralls

veruu
veruu approved these changes May 21, 2018
View reviewed changes
Copy link
Contributor

@veruu veruu left a comment

Choose a reason for hiding this comment

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

LGTM, hopefully I didn't miss anything :)

@spbnick spbnick merged commit 783b5fe into cki-project:master May 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@veruu veruu veruu approved these changes

Assignees

@major major

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@major @coveralls @veruu @spbnick