Skip to content
This repository has been archived by the owner on Oct 1, 2020. It is now read-only.

removing files unused in this project #674

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

marcosptf
Copy link
Collaborator

@marcosptf marcosptf commented Nov 23, 2018

after reviewing some codes in this project, i have saw that this files is not using, then to keep the code clean, i would like to suggest remove then;

Thanks

Att,
@marcosptf

@marcosptf marcosptf changed the title removing files unused in this project WIP: removing files unused in this project Nov 23, 2018
@pep8speaks
Copy link

pep8speaks commented Jun 6, 2019

Hello @marcosptf! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 46:47: E231 missing whitespace after ':'
Line 46:56: E231 missing whitespace after ':'
Line 46:65: E231 missing whitespace after ':'
Line 46:74: E231 missing whitespace after ':'
Line 46:80: E501 line too long (88 > 79 characters)
Line 46:83: E231 missing whitespace after ':'
Line 47:76: E231 missing whitespace after ':'
Line 47:80: E501 line too long (117 > 79 characters)
Line 47:85: E231 missing whitespace after ':'
Line 47:94: E231 missing whitespace after ':'
Line 47:103: E231 missing whitespace after ':'
Line 47:112: E231 missing whitespace after ':'
Line 53:80: E501 line too long (127 > 79 characters)
Line 61:80: E501 line too long (126 > 79 characters)
Line 69:80: E501 line too long (137 > 79 characters)
Line 77:80: E501 line too long (141 > 79 characters)
Line 85:80: E501 line too long (149 > 79 characters)
Line 86:80: E501 line too long (83 > 79 characters)
Line 93:80: E501 line too long (135 > 79 characters)
Line 94:45: W291 trailing whitespace
Line 104:80: E501 line too long (126 > 79 characters)
Line 105:45: W291 trailing whitespace
Line 115:80: E501 line too long (136 > 79 characters)
Line 116:45: W291 trailing whitespace
Line 126:80: E501 line too long (149 > 79 characters)
Line 127:45: W291 trailing whitespace
Line 130:71: E231 missing whitespace after ':'
Line 130:80: E231 missing whitespace after ':'
Line 130:80: E501 line too long (112 > 79 characters)
Line 130:89: E231 missing whitespace after ':'
Line 130:98: E231 missing whitespace after ':'
Line 130:107: E231 missing whitespace after ':'
Line 137:80: E501 line too long (148 > 79 characters)
Line 138:45: W291 trailing whitespace
Line 141:71: E231 missing whitespace after ':'
Line 141:80: E231 missing whitespace after ':'
Line 141:80: E501 line too long (112 > 79 characters)
Line 141:89: E231 missing whitespace after ':'
Line 141:98: E231 missing whitespace after ':'
Line 141:107: E231 missing whitespace after ':'
Line 148:80: E501 line too long (139 > 79 characters)
Line 149:45: W291 trailing whitespace
Line 152:71: E231 missing whitespace after ':'
Line 152:80: E231 missing whitespace after ':'
Line 152:80: E501 line too long (112 > 79 characters)
Line 152:89: E231 missing whitespace after ':'
Line 152:98: E231 missing whitespace after ':'
Line 152:107: E231 missing whitespace after ':'

Comment last updated at 2019-06-06 10:32:34 UTC

@marcosptf marcosptf changed the title WIP: removing files unused in this project removing files unused in this project Jun 6, 2019
Copy link

@cesarbruschetta cesarbruschetta left a comment

Choose a reason for hiding this comment

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

I just believe it is good to make a git squash to gather all commits before sending to the master

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants