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

Minor refactor #66

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

DJStompZone
Copy link

Refactor and update project dependencies

  • Removed unused using directives across multiple files.
  • Simplified object instantiation and using statements.
  • Changed ImageIcon to a readonly struct for immutability.
  • Simplified dictionary initialization with new C# syntax.
  • Migrated from WebClient to HttpClient in Updater class.
  • Updated project files to target newer frameworks and tools.
  • Updated Newtonsoft.Json package to version 13.0.3.
  • Marked submodules fNbt and utils.forms as dirty.
  • Removed redundant and commented-out code.
  • Used nameof for argument names in exception handling.
  • Refactored switch statements to use pattern matching.
  • Removed duplicate PackageReference from project file.

- Removed unused `using` directives across multiple files.
- Simplified object instantiation and using statements.
- Changed `ImageIcon` to a `readonly struct` for immutability.
- Simplified dictionary initialization with new C# syntax.
- Migrated from `WebClient` to `HttpClient` in `Updater` class.
- Updated project files to target newer frameworks and tools.
- Updated `Newtonsoft.Json` package to version `13.0.3`.
- Marked submodules `fNbt` and `utils.forms` as dirty.
- Removed redundant and commented-out code.
- Used `nameof` for argument names in exception handling.
- Refactored switch statements to use pattern matching.
- Removed duplicate `PackageReference` from project file.
Updated the subproject commit in the `fNbt` directory from
commit `2be4eaa701aeedda2c2edd8052d46cc3eaca625a` to commit
`35de13a94c6f566bd518916f40c6f5d768222b90`.

Updated the subproject commit in the `utils.forms` directory
from commit `8ed4ce8f05a512a76c80519d49738e7d0745f929` to commit
`ed194dcd7e4128538fac33dc49b1e14685a6a8e1`.
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.

1 participant