Skip to content

Conversation

@bashir2
Copy link
Member

@bashir2 bashir2 commented Dec 6, 2018

Tested: Ran deploy_and_run_tests.sh.

@bashir2 bashir2 requested a review from allieychen December 6, 2018 18:17
@allieychen
Copy link
Contributor

Thank you Bashir! But I think Saman already has a PR that includes this change. :)

@bashir2
Copy link
Member Author

bashir2 commented Dec 7, 2018

Oh, okay, so I close this and let @samanvp fix this by his PR; thanks both.

@bashir2 bashir2 closed this Dec 7, 2018
@samanvp
Copy link
Contributor

samanvp commented Dec 7, 2018

I just submitted #416
Thank you both

@coveralls
Copy link

Pull Request Test Coverage Report for Build 1469

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 87.639%

Totals Coverage Status
Change from base Build 1467: 0.0%
Covered Lines: 6445
Relevant Lines: 7354

💛 - Coveralls

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.

4 participants