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

Add script for removing whitespaces #41

Closed
aktech opened this issue Apr 5, 2016 · 13 comments
Closed

Add script for removing whitespaces #41

aktech opened this issue Apr 5, 2016 · 13 comments

Comments

@aktech
Copy link
Member

aktech commented Apr 5, 2016

No description provided.

@a2batic
Copy link
Contributor

a2batic commented Apr 7, 2016

From where should these white spaces be deleted?

@souravsingh
Copy link

Whitespace removal could be done by autopep8, I think. Or are you thinking of something like this-
http://paste.fedoraproject.org/355165/5778081/

@aktech
Copy link
Member Author

aktech commented Apr 13, 2016

From where should these white spaces be deleted?

The extra whitespaces in any file.

@aktech
Copy link
Member Author

aktech commented Apr 13, 2016

You may take motivation from this:
https://github.com/sympy/sympy/blob/master/bin/strip_whitespace

@a2batic
Copy link
Contributor

a2batic commented Apr 14, 2016

I want to take up this task, please assign it to me.

@CuriousLearner
Copy link
Member

@souravsingh Don't use new as a variable name in your code.

@kanikaa1234 go ahead.

@souravsingh
Copy link

@CuriousLearner It is just a small sample code which I wrote. I know about the thing for variable name.

@CuriousLearner
Copy link
Member

@souravsingh I meant whatever code you write, even if it's a sample code. It is not a good practice to use such variable names :)

@a2batic
Copy link
Contributor

a2batic commented Apr 18, 2016

We have made a PR ( #70 ) for this issue.

a2batic added a commit to a2batic/pydsa that referenced this issue Apr 24, 2016
@a2batic
Copy link
Contributor

a2batic commented Apr 24, 2016

We have made the changes in PR ( #72 ).

@rahulroxx
Copy link
Contributor

rahulroxx commented May 12, 2016

#85 @aktech @CuriousLearner I think that we should add pylint support in Travis CI , Let Pylint handle all the issues.

@a2batic
Copy link
Contributor

a2batic commented May 14, 2016

@rahulroxx, but Pylint will not remove the trailing spaces !!

@rahulroxx
Copy link
Contributor

rahulroxx commented May 14, 2016

@kanikaa1234 Agreed ! It was meant for PR's

flake8 is doing the same so, I closed the issue. Codebase is reformatted in #89 and #84

rahulroxx pushed a commit to rahulroxx/pydsa that referenced this issue May 20, 2016
* Fixes pydsa#41

* Added doc string and code changes

* Addressing changes

;
q'

* Correcting spelling
prachiagrawal269 pushed a commit to prachiagrawal269/pydsa that referenced this issue May 21, 2016
* Fixes pydsa#41

* Added doc string and code changes

* Addressing changes

;
q'

* Correcting spelling
prachiagrawal269 pushed a commit to prachiagrawal269/pydsa that referenced this issue May 21, 2016
Reformatted radix_sort.py and some docs

Add Rehas and Rahul to Authors

Fixes pydsa#41 (pydsa#72)

* Fixes pydsa#41

* Added doc string and code changes

* Addressing changes

;
q'

* Correcting spelling

Add Binary Tree implementation, tests

Modify delete function and add docstrings to binary_tree

Add Kanika to AUTHORS. Welcome to PyDSA!

Added Gnome Sort

Remove import *

Squashed commits

Nameerror removed

Squashed commits
prachiagrawal269 pushed a commit to prachiagrawal269/pydsa that referenced this issue May 21, 2016
Reformatted radix_sort.py and some docs

Add Rehas and Rahul to Authors

Fixes pydsa#41 (pydsa#72)

* Fixes pydsa#41

* Added doc string and code changes

* Addressing changes

;
q'

* Correcting spelling

Add Binary Tree implementation, tests

Modify delete function and add docstrings to binary_tree

Add Kanika to AUTHORS. Welcome to PyDSA!

Added Gnome Sort

Remove import *

Squashed commits

Nameerror removed

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

No branches or pull requests

5 participants