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

Add port parameter to local command #220

Merged
merged 1 commit into from
Jan 26, 2017
Merged

Add port parameter to local command #220

merged 1 commit into from
Jan 26, 2017

Conversation

gliptak
Copy link
Contributor

@gliptak gliptak commented Jan 7, 2017

Please check closely as my local dev environment is not fully running ...

@codecov-io
Copy link

codecov-io commented Jan 7, 2017

Current coverage is 70.46% (diff: 11.11%)

Merging #220 into master will decrease coverage by 0.14%

@@             master       #220   diff @@
==========================================
  Files            12         12          
  Lines          1480       1483     +3   
  Methods           0          0          
  Messages          0          0          
  Branches        191        191          
==========================================
  Hits           1045       1045          
- Misses          384        387     +3   
  Partials         51         51          

Powered by Codecov. Last update 79560c8...37ba3b1

@gamame
Copy link

gamame commented Jan 11, 2017

+1

@andriisoldatenko
Copy link
Contributor

@gliptak Can you please add unit tests?
will decrease coverage by 0.14%

@gliptak
Copy link
Contributor Author

gliptak commented Jan 12, 2017

@andriisoldatenko Would you offer a recommendation which unit test to clone from? Thanks

@gliptak
Copy link
Contributor Author

gliptak commented Jan 19, 2017

Any guidance? Thanks

@jamesls
Copy link
Member

jamesls commented Jan 26, 2017

Thanks for the pull request and sorry for the delay. I've gone ahead and added a test so we can get this merged.

@jamesls jamesls merged commit 37ba3b1 into aws:master Jan 26, 2017
@gliptak gliptak deleted the port1 branch January 27, 2017 01:56
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.

5 participants