Skip to content
This repository has been archived by the owner on Dec 12, 2024. It is now read-only.

[style] Fix up style in Inventory namespace #797

Open
stephen-hawley opened this issue Nov 28, 2023 · 0 comments
Open

[style] Fix up style in Inventory namespace #797

stephen-hawley opened this issue Nov 28, 2023 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers
Milestone

Comments

@stephen-hawley
Copy link
Contributor

Look over code and make changes including:

  • using var where it makes sense
  • clean up type-checking using the if (cl is SomeType varName) pattern
  • remove SomeType varName = null; if (values.TryGet(name, out varName) pattern and use if (values.TryGet(name, out varVarname))
  • Turn on #nullable enable
@stephen-hawley stephen-hawley added enhancement New feature or request good first issue Good for newcomers labels Nov 28, 2023
@stephen-hawley stephen-hawley added this to the Future milestone Nov 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant