Skip to content

Conversation

@eamanu
Copy link
Contributor

@eamanu eamanu commented Apr 8, 2017

Create de Remove specified characters in a string and its solution and testcases.

Send the changes to review

Regards!

@donnemartin
Copy link
Owner

Hi @eamanu, thanks for the PR! I hope to review it within the next few days.

@donnemartin
Copy link
Owner

Hi @eamanu, this is a great start!

I think the use of string.replace simplifies the problem considerably, can you think of a way to solve it without using this built-in method?

Please also check out the Maintain Consistent Style section of the Contributing Guidelines (mainly the PEP8 link).

Thanks!

@eamanu
Copy link
Contributor Author

eamanu commented Apr 9, 2017

HI @donnemartin!

  • I make a new algorithm and remove the string.replace.
  • I correct the PEP8 style.

Please, review it.
Regards!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants