Skip to content

Conversation

basakest
Copy link
Contributor

@basakest basakest commented Apr 1, 2021

No description provided.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 707786590

  • 0 of 18 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+4.6%) to 76.364%

Totals Coverage Status
Change from base Build 705511255: 4.6%
Covered Lines: 84
Relevant Lines: 110

💛 - Coveralls

@coveralls
Copy link

coveralls commented Apr 1, 2021

Pull Request Test Coverage Report for Build 708073656

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 19 of 19 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+5.7%) to 77.391%

Totals Coverage Status
Change from base Build 708034362: 5.7%
Covered Lines: 89
Relevant Lines: 115

💛 - Coveralls

@Player626 Player626 self-requested a review April 1, 2021 09:43
@hsluoyz hsluoyz requested a review from leeqvip April 1, 2021 10:26
@hsluoyz
Copy link
Member

hsluoyz commented Apr 1, 2021

@techoner plz review.

@Player626
Copy link
Collaborator

The merger can be carried out.

@hsluoyz hsluoyz merged commit 83ac41c into php-casbin:master Apr 1, 2021
@leeqvip
Copy link
Member

leeqvip commented Apr 1, 2021

🎉 This PR is included in version 1.2.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants