-
-
Notifications
You must be signed in to change notification settings - Fork 34
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Convenience Constructors to shapes #284
Conversation
/// <summary> | ||
/// Initializes a new instance of the <see cref="Path"/> class. | ||
/// </summary> | ||
/// <param name="points">The collection of points.</param> |
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.
worth pointing out these are processed as a series of linear line segments
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.
done
@@ -11,6 +11,15 @@ namespace SixLabors.ImageSharp.Drawing | |||
/// </summary> | |||
public class Polygon : Path | |||
{ | |||
/// <summary> | |||
/// Initializes a new instance of the <see cref="Polygon"/> class. |
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.
again woth pointing out thye are handled as a LinearLineSegment
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.
and done
Codecov Report
@@ Coverage Diff @@
## main #284 +/- ##
===================================
Coverage 80% 80%
===================================
Files 97 97
Lines 5020 5024 +4
Branches 919 919
===================================
+ Hits 4025 4029 +4
Misses 796 796
Partials 199 199
Flags with carried forward coverage won't be shown. Click here to find out more.
馃摚 We鈥檙e building smart automated test selection to slash your CI/CD build times. Learn more |
Prerequisites
Description
Fix #100