Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Code Refactoring #33

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Code Refactoring #33

wants to merge 1 commit into from

Conversation

arpt-svt
Copy link
Contributor

Fixes linting issue, var names, using context management for opening and closing file
Signed-off-by: ARPIT SRIVASTAVA arpit.s@mistay.in

Code Refactors: Fixes linting issue, var names, using context managem…
86cb975 
…ent for opening and closing file

Signed-off-by: ARPIT SRIVASTAVA <arpit.s@mistay.in>
@arpt-svt arpt-svt mentioned this pull request Oct 13, 2018
Open
ibreakthecloud
ibreakthecloud reviewed Oct 15, 2018
View reviewed changes
Copy link
Contributor

@ibreakthecloud ibreakthecloud left a comment

Choose a reason for hiding this comment

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

Can you add few comments in all functions?

# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@ibreakthecloud ibreakthecloud ibreakthecloud left review comments

Assignees
No one assigned
Labels
Hacktoberfest
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@arpt-svt @ibreakthecloud @ksatchit