search

YahooArchive / arrow

FE Test framework designed to promote TDD
http://yahoo.github.io/arrow/arrow_intro.html
BSD 3-Clause "New" or "Revised" License
55 stars 59 forks source link

change var assignments to single statment #26

Closed rynodivino closed 11 years ago

rynodivino commented 11 years ago

These are purely style changes, and no logic affected. So submitting them in their own code request to make it easy to ignore if hunting logic changes.

Oh, changing these to follow normal coding convention (which is even used further down throughout most of these same files - just for some reason the vars declared at top of files were all done individually).

proverma commented 11 years ago

@rynodivino I am already working on this. I have made lot of changes in my local version, hence I wont be able to take this pull request, but be assured ...we will be jslint error free with my next check-in. Thanks for looking into it though.

rynodivino commented 11 years ago

Ah, cool. I'll close the pull request.

Is there any chance that when you're developing, and have lots of changes. Before you are ready to push a new version, why not be committing to a "work in progress" branch. So if other dev's are working on a new feature, they can we working with your branch, and it will not be so difficult to merge at the end. What do you think?