Daniela Meneses wrote:
Hi to all,

I'm writing missing test for the String class methods to improve the test coverage. I will commit my changes in this project at SmalltalkHub:


Feel free to take a look. Any feedback will be really welcome and useful. :D

--
Cheers,
Daniela
hi Daniela,

That is a nice initiative.  I guess you have bigger plans to "rethink the String manipulation API of Pharo" where you will want to be experimenting in a separate branch, but adding tests would be beneficial for the mainline now.  I'd encourage you to open a Case at pharo.fogbugz.com and submit those tests as a slice :)
You definitely get feedback as part of the integration review process.
cheers -ben

Reply via email to