-
Notifications
You must be signed in to change notification settings - Fork 610
Add a rect shape with corners having different radii #3991
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #3991 +/- ##
==========================================
+ Coverage 48.37% 48.59% +0.21%
==========================================
Files 194 195 +1
Lines 22400 22447 +47
==========================================
+ Hits 10837 10909 +72
+ Misses 11563 11538 -25
|
I'm not sure why the test failed. |
Codecov fails because none of your lines are covered by tests and you end up below the target. That's another thing documentation examples are used for: they also feed into the testing framework. |
Thanks for the help. I also added tests. |
@Aire-One Hi! Aren't there any other reviewers to approve my PR? |
Hi @aidinio, sadly most other maintainers are AWOL for some time now... Let's try to ping @awesomeWM/write-access |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
but coverage is not enough: #3991 (comment)
I've already added tests which resolve the coverage issue. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, codecov stuck on previous commit: https://app.codecov.io/gh/awesomeWM/awesome/pull/3991/commits
Can I somehow restart the codecov check? It's stuck on "Waiting for status to be reported". |
no, as CI currently broken: #3994 |
Added a shape that can have a different radius on each corner.