Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

A bunch of minor fixes #60

Merged
merged 6 commits into from Aug 8, 2014
Merged

A bunch of minor fixes #60

merged 6 commits into from Aug 8, 2014

Conversation

anton-rudeshko
Copy link
Contributor

  • Added IntelliJ IDEA workspace to .gitignore
  • Fixed mixed tabs and spaces (replaced tabs with spaces)
  • Fixed typo
  • Fixed inappropriate JSDoc params
  • Added missing semicolons and proper indent

@anton-rudeshko
Copy link
Contributor Author

Ping

@lucasvo
Copy link
Collaborator

lucasvo commented Aug 8, 2014

@anton-rudeshko The changes look good, thanks for cleaning up a bit!

lucasvo added a commit that referenced this pull request Aug 8, 2014
A bunch of minor fixes

Added IntelliJ IDEA workspace to .gitignore
Fixed mixed tabs and spaces (replaced tabs with spaces)
Fixed typo
Fixed inappropriate JSDoc params
Added missing semicolons and proper indent
@lucasvo lucasvo merged commit 3d757d4 into steves:master Aug 8, 2014
@anton-rudeshko anton-rudeshko deleted the minors branch August 9, 2014 07:37
@anton-rudeshko
Copy link
Contributor Author

Finally someone got a look at this =) Thank you

@lucasvo
Copy link
Collaborator

lucasvo commented Aug 9, 2014

I will still have to build a new version. Let me do that soon!

On Aug 9, 2014, at 12:37 AM, "Anton Rudeshko (Tesla)" notifications@github.com wrote:

Finally someone got to look at this =) Thank you


Reply to this email directly or view it on GitHub.

@lucasvo-taulia
Copy link
Contributor

Thanks for your pull request. Version 0.9.2 has been released with your fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants