Navigation Menu

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

Syntax improvements #48

Merged
merged 3 commits into from Jan 20, 2017
Merged

Syntax improvements #48

merged 3 commits into from Jan 20, 2017

Conversation

agourlay
Copy link
Contributor

Realised that I forgot to format the test code, so I did that plus a couple of syntax improvements on top.

Don't worry, no more PR for today ;)

@codecov-io
Copy link

codecov-io commented Jan 19, 2017

Current coverage is 92.03% (diff: 100%)

Merging #48 into master will not change coverage

@@             master        #48   diff @@
==========================================
  Files            11         11          
  Lines           502        502          
  Methods         460        460          
  Messages          0          0          
  Branches         42         42          
==========================================
  Hits            462        462          
  Misses           40         40          
  Partials          0          0          

Powered by Codecov. Last update 11f1a32...7a26ca0

@satabin
Copy link
Member

satabin commented Jan 19, 2017

On the contrary, I like having PRs, it means the project is of interest :) Thanks for all your contributions

@satabin satabin self-assigned this Jan 20, 2017
@satabin satabin modified the milestones: 2.1.1, 2.1.2 Jan 20, 2017
@satabin satabin merged commit f9705f5 into gnieh:master Jan 20, 2017
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.

None yet

3 participants