Skip to content
New issue

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

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

Already on GitHub? # to your account

openssl_encrypt with no 5th argument is failing #93

Merged
merged 6 commits into from
Mar 7, 2019

Conversation

moufmouf
Copy link
Member

@moufmouf moufmouf commented Feb 2, 2019

See #76.

PR starts with a failing test.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 163

  • 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 35.436%

Totals Coverage Status
Change from base Build 160: 0.0%
Covered Lines: 191
Relevant Lines: 539

💛 - Coveralls

1 similar comment
@coveralls
Copy link

Pull Request Test Coverage Report for Build 163

  • 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 35.436%

Totals Coverage Status
Change from base Build 160: 0.0%
Covered Lines: 191
Relevant Lines: 539

💛 - Coveralls

@coveralls
Copy link

coveralls commented Feb 2, 2019

Pull Request Test Coverage Report for Build 177

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 19 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.2%) to 35.78%

Files with Coverage Reduction New Missed Lines %
src/Method.php 19 46.24%
Totals Coverage Status
Change from base Build 176: 0.2%
Covered Lines: 195
Relevant Lines: 545

💛 - Coveralls

@moufmouf moufmouf merged commit 67b654c into thecodingmachine:master Mar 7, 2019
@moufmouf moufmouf deleted the openssl_encrypt branch March 7, 2019 11:09
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants