Skip to content

Minor refactor#66

Open
DJStompZone wants to merge 2 commits into
tryashtar:masterfrom
DJStompZone:minor-refactor
Open

Minor refactor#66
DJStompZone wants to merge 2 commits into
tryashtar:masterfrom
DJStompZone:minor-refactor

Conversation

@DJStompZone
Copy link
Copy Markdown

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