Skip to content

Update ReadMe#258

Open
Happypig375 wants to merge 4 commits intomasterfrom
update-readme
Open

Update ReadMe#258
Happypig375 wants to merge 4 commits intomasterfrom
update-readme

Conversation

@Happypig375
Copy link
Collaborator

@Happypig375 Happypig375 commented Feb 26, 2026

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.

@Happypig375
Copy link
Collaborator Author

@charlesroddie Review this and #159 thanks

ReadMe.md Outdated
[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
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, yea.


To get started, do something like this:

### 1. CSharpMath.SkiaSharp
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why remove these instructions?

Copy link
Collaborator Author

@Happypig375 Happypig375 Mar 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

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