Skip to content
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

Feature/upstream #2

Merged
merged 90 commits into from
Jun 2, 2024
Merged

Feature/upstream #2

merged 90 commits into from
Jun 2, 2024

Conversation

jeremj22
Copy link
Owner

No description provided.

Lilith Silver and others added 30 commits September 3, 2023 23:57
Conversion from circle to rectangle should not pass unnoted and be explicit.
In that way the OrientedBoundedRectangle can take part of collision tests.
…or_undefined_layer

Check for undefined layer and throw on collision actor insertion
…nding-rectangle

Add oriented rectangle shape for improved collision detection
…etpressedkeys

Decrease allocation overhead on KeyboardStateExtended
Improve naming of WasKeyJustDown/Up and WasButtonJustDown/Up
When both params `a` and `b` are of type `RectangleF` just calling `Shape.Intersects` creates a infinite recursion bug.
…ewtonsoft-dependency

Update Newtonsoft.Json.dll Dependency
…cs-recursion-bug

Resolves Infinite Recursion with Shape.Intersects(this IShapeF, IShapeF)
Lithiums MyGet feed won't allow the NuGet push due to the SemVersions of the package containing a 4th build number value.
Moved benchmark and tweening test projects into the `/src/cs` to keep project organization constancy.
AristurtleDev and others added 25 commits May 19, 2024 16:12
* Fix github url for nuget push

* forgot the h in https
* Fix github url for nuget push

* forgot the h in https

* Remove github push for now, need to check repository settings
* Removed Roadmap
The Roadmap seems to have been initially added to show the plans to switch to an FNA fork and act as a bridge for katabasis.  This is no longer the case for this repository as it will remain focused at the moment on MonoGame support only.

* Added new note at top of readme

* updated information for installing nuget

* Made using content pipeline extensions a section and not a quote

* Removed information about MyGet feed

* Removed patreon from readme

* Update License

* renamed master branch to main

* Remove community forum link
MonoGame is closing the community forums in favor of discord and github discussions

* Update workflows
* Converted to use `System.Text.Json`

* Remove Newtonsoft.Json Dependency
Newtonsoft.Json dependency has been removed in favor of System.Text.Json

* Treat MGFXO file as binary
* Replace `Matrix2` with `Matrix3x2`

* Update benchmarks

* Add XML documentation for `Matrix3x2`
* Added Assert for edge contains
Reference: craftworkgames#214

* Renamed to PolygonTests
* Added `CalculateMinimumVector2` and `CalculateMaximumVector2` as replacement for `Point2.Minimum` and `Point2.Maximum`

* Remove `Point2`
@jeremj22 jeremj22 self-assigned this May 30, 2024
@jeremj22 jeremj22 requested a review from ferafda May 30, 2024 07:41
@jeremj22
Copy link
Owner Author

Undo downgrade of Newtonsoft.Json and instead pull from the main repo where it's no longer being used. Gets rid of the vulnerability notice you probably saw in your IDE since the original upgrade was done to avoid it.

Quite a bit to review. Best look only at the commits. I had to undo some of their commits because it resulted in different paths for release and debug DLLs which I don't believe we can properly target from the Content.mgcb

@jeremj22 jeremj22 merged commit c097ce2 into develop Jun 2, 2024
2 of 3 checks passed
@jeremj22 jeremj22 deleted the feature/upstream branch June 2, 2024 20:31
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.

6 participants