Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR moves the ServerState and GlobalState types into it's own crate
Description
The definitions for the GlobalState and ServerState have been moved into it's own crate since it made very little sense to have it in the net crate. On the surface this seems like it would cause cyclic dependency issues but it won't actually introduce any more issues than having it in the net crate, other than preventing the net crate from providing types that are used in the state, but it wasn't doing that in the first place.
Motivation and Context
This change was implemented to clear up confusion regarding having the state implemented in the net crate.
How has this been tested?
Since it's just moving definitions from one place to another, no unit tests are required because if it compiles, it works.
Screenshots (if appropriate):
Types of changes
Checklist: