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

refactor: readers no longer depends on NetworkClient and NetworkServer #583

Merged
merged 3 commits into from
Feb 14, 2021

Conversation

paulpach
Copy link
Contributor

NetworkReader no longer has a hardcoded dependency on NetworkClient and NetworkServer.
rather the NetworkReader has an IObjectLocator, which can be any class
that can find objects by netid.

BREAKING CHANGE: NetworkReader no longer have .Client and .Server, it has a .ObjectLocator instead

@paulpach paulpach changed the title refactor: NetworkReader no longer depends on NetworkClient and NetworkServer refactor: readers no longer depends on NetworkClient and NetworkServer Feb 14, 2021
…kServer

NetworkReader no longer has a hard coded dependency on NetworkClient and NetworkServer.
rather the NetworkReader has an IObjectLocator, which can be any class
that can find objects by netid.

BREAKING CHANGE: NetworkReader no longer have .Client and .Server, it has a .ObjectLocator instead
@sonarcloud
Copy link

sonarcloud bot commented Feb 14, 2021

@paulpach paulpach merged commit 177c307 into master Feb 14, 2021
@paulpach paulpach deleted the locator branch February 14, 2021 22:20
github-actions bot pushed a commit that referenced this pull request Feb 14, 2021
# [65.0.0](v64.0.0...v65.0.0) (2021-02-14)

### Code Refactoring

* NetworkReader no longer depends on NetworkClient and NetworkServer ([15f27c4](15f27c4))
* readers no longer depends on NetworkClient and NetworkServer ([#583](#583)) ([177c307](177c307))

### BREAKING CHANGES

* NetworkReader no longer have .Client and .Server, it has a .ObjectLocator instead
* NetworkReader no longer have .Client and .Server, it has a .ObjectLocator instead
@github-actions
Copy link
Contributor

🎉 This PR is included in version 65.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants