Update ReadMe#258
Conversation
|
@charlesroddie Review this and #159 thanks |
| [NuGet-pre]: https://www.nuget.org/packages/CSharpMath/absoluteLatest | ||
| [GitHub]: https://github.com/verybadcat/CSharpMath/releases/latest | ||
| [GitHub-pre]: https://github.com/verybadcat/CSharpMath/releases | ||
| [GitHub-pre]: https://github.com/verybadcat/CSharpMath/releas/latest |
|
|
||
| To get started, do something like this: | ||
|
|
||
| ### 1. CSharpMath.SkiaSharp |
There was a problem hiding this comment.
Why remove these instructions?
There was a problem hiding this comment.
It is replaced by the link to the Example file but I also see how including some description on the main type to use is also wanted. I'll add it.
There was a problem hiding this comment.
This mess (with some projects referencing core and core referencing projects outside) is something to fix. So this image could go into an issue to track this, not the repo itself.
There was a problem hiding this comment.
@charlesroddie I changed the solution structure. Does it look better?
There was a problem hiding this comment.
|
@charlesroddie review please |
|
I'm sorry for the delay in replying. This PR may be an improvement but there are things added which should not be in the main README.md as they are not relevant to end-users. Projects.png is not relevant. It's also an image, which should be converted to text. I think there should be another .md file which is relevant to contributors or developers. I can help to split this up if that would help. |
2 participants
Related to #140
The current solution structure mostly flows from top to bottom just like F# code. I prefer this over forcibly grouping projects into Tests and Examples folders for example.