Skip to content

Conversation

@JacksonJang
Copy link
Contributor

Issue: #5405
I've fixed the issue.

Additionally, since the test without b had been removed, I added that test case back as well.

@JacksonJang
Copy link
Contributor Author

I updated the patch so that the property path simply calls _checkShapeShifting(...) directly.

@JacksonJang
Copy link
Contributor Author

@cowtowncoder
Would it be possible to consider merging this issue as well?

@cowtowncoder
Copy link
Member

@JacksonJang I'll try to review this soon. Thank you!

@cowtowncoder
Copy link
Member

Note: Trying to get CI code coverage working for external contributors.

@github-actions
Copy link

🧪 Code Coverage Report

Metric Coverage
Instructions coverage 78.7%
Branches branches 72.5%

Coverage data generated from JaCoCo test results

@github-actions
Copy link

🧪 Code Coverage Report

Metric Coverage Change
Instructions coverage 78.7% 📈 +0.0%
Branches branches 72.5% 📈 +0.0%

Coverage data generated from JaCoCo test results

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.

2 participants