Skip to content

Conversation

@nowsprinting
Copy link
Contributor

@nowsprinting nowsprinting commented Jul 2, 2025

The NuGet package needs to respect a few constraints in order to be listed in the curated list:

  • Add a link to the NuGet package: https://www.nuget.org/packages/VYaml https://www.nuget.org/packages/VYaml.Annotations
  • It must have non-preview versions (e.g 1.0.0 but not 1.0.0-preview.1)
  • It must provide .NETStandard2.0 assemblies as part of its package
  • The lowest version added must be the lowest .NETStandard2.0 version available
  • The package has been tested with the Unity editor
  • The package has been tested with a Unity standalone player
    • if the package is not compatible with standalone player, please add a comment to a Known issues section to the top level readme.md
  • All package dependencies with .NETStandard 2.0 target must be added to the PR (respecting the same rules above)
    • Note that if a future version of the package adds a new dependency, this dependency will have to be added manually as well

Note: The server will be updated only when a new version tag is pushed on the main branch, not necessarily after merging this pull-request.

@bdovaz bdovaz enabled auto-merge (squash) July 2, 2025 07:24
@nowsprinting
Copy link
Contributor Author

2025-07-02T08:55:42.2429566Z   Failed Ensure_Min_Version_Is_Correct_Ignoring_Analyzers_And_Native_Libs(VYaml.Annotations, [1.1.0, )) [9 ms]
2025-07-02T08:55:42.2453467Z   Error Message:
2025-07-02T08:55:42.2468797Z      Package VYaml.Annotations
2025-07-02T08:55:42.2469543Z Assert.That(versionRange!.MinVersion, Is.EqualTo(packageIdentity.Version))
2025-07-02T08:55:42.2469918Z   Expected: <0.28.0>
2025-07-02T08:55:42.2470091Z   But was:  <1.1.0>

auto-merge was automatically disabled July 2, 2025 16:27

Head branch was pushed to by a user without write access

@bdovaz bdovaz merged commit 6d991fe into bdovaz:master Jul 2, 2025
3 checks passed
@nowsprinting nowsprinting deleted the vyaml branch July 3, 2025 00:57
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