benkasminbullock / json-create

https://metacpan.org/release/JSON-Create
1 stars 4 forks source link

removing TODO notes that already are implemented #39

Closed joenio closed 8 years ago

benkasminbullock commented 8 years ago

It's OK to remove these I suppose. I'm not sure why you need to do that though, also with the change from tabs to spaces, it doesn't seem like something which someone not working on the code would really need to do. At the moment I'm the only person working on this codebase, and it doesn't really help me that much. Changing tabs to spaces doesn't help me at all, I use Emacs and those kinds of whitespace differences are irrelevant to me. If you want to work on the codebase, perhaps you could do that job and send changes as pull requests. If this is for the pull request challenge, I'd rather you did something else which might benefit the module users, rather than mess around with comments and whitespace, which doesn't benefit the module users, and doesn't benefit the programmer (which is me alone).