Skip to content

Conversation

@ShadauxCat
Copy link
Collaborator

No description provided.

* update

excluding getTypeName,

* update

Make sure internal classes can access NetworkBehaviour.NetworkVariableFields in runtime builds.

* update

to get RPCS working temporarily.

* style

Adding comments about the RPC stuff since it is a bit more complicated than the previous two issues.

* style

updating and adding comments to regions of code changed in order to easily distinguish the changes made for each issue type.

* update

Missed two files.

* Updated the ILPP code to work correctly with CoreCLR:
- delegates are no longer fields, but types
- internal virtual functions can't be overridden in child classes anymore, so they have to be changed from `IsAssembly` to `IsFamilyOrAssembly` in ILPP
- Internal fields changed to `IsFamily` can no longer be accessed within the assembly, and need to be `IsFamilyOrAssembly` instead.

With those changes, previous workarounds are no longer needed, so I reverted them.

* update

removing commented out code.
wrapping the rpc table generation log info so it only logs to the console when log mode is set to developer.

* style

Fixing whitespace issue.

* revert __RpcParams to internal

* update

adding change log entry

* update

Moving the version number back down to v1.5.1

* Fixed validator failures.

* Update package.json

Reverting back to v1.5.2 to validate compatibility testing

* Add Validation exceptions for 1.5.2

* Apparently the meta file is also needed.

---------

Co-authored-by: Kitty Draper <kitty.draper@unity3d.com>
@ShadauxCat ShadauxCat requested a review from a team as a code owner July 21, 2023 20:04
@ShadauxCat ShadauxCat enabled auto-merge (squash) July 21, 2023 20:07
@ShadauxCat ShadauxCat merged commit 3394881 into release/1.5.2 Jul 21, 2023
NoelStephensUnity added a commit that referenced this pull request Aug 1, 2023
* merging develop into release/1.5.2 (#2634)

* chore: tagging 1.5.2 in the changelog (#2640)

---------

Co-authored-by: Kitty Draper <284434+ShadauxCat@users.noreply.github.com>
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