Skip to content

Conversation

andrej-griniuk
Copy link
Contributor

No description provided.

@coveralls
Copy link

coveralls commented Jun 14, 2019

Coverage Status

Coverage remained the same at 99.218% when pulling 9ade739 on andrej-griniuk:feature/add-client-options-to-constructor into 40ef1fc on hyperwallet:master.

@andrej-griniuk
Copy link
Contributor Author

Any chance to get this merged or is there anything else that needs to be done?

# Conflicts:
#	src/Hyperwallet/Hyperwallet.php
@specialtactics
Copy link
Contributor

Hey @wmews-hw any reason this hasn't been merged yet ?

@andrej-griniuk
Copy link
Contributor Author

Not sure why the build is failing in PHP 5.6 while passing in 7.0 and HHVM. Seems unrelated to my changes.

specialtactics added a commit to specialtactics/php-sdk that referenced this pull request Oct 7, 2020
@specialtactics
Copy link
Contributor

I'd like to reference this short investigation about the failing test:
#43

@specialtactics
Copy link
Contributor

specialtactics commented Oct 7, 2020

I made a PR to fix the failing test here: #44

Please merge that PR, and then you can merge this. Although this PR doesn't cause any tests to fail anyway - that test just fails randomly.

@specialtactics
Copy link
Contributor

specialtactics commented Oct 7, 2020

@Rrathinasabapath could you please look at this also?

@arao6 arao6 requested review from a user and peter-joseph October 16, 2020 20:05
@arao6 arao6 changed the base branch from master to V4 October 19, 2020 18:10
@akalichety-hw akalichety-hw merged commit fc12496 into hyperwallet:V4 Oct 20, 2020
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.

6 participants