Fix a reportUnnecessaryIsInstance Pyright warning

[correctmost]

PR Description:

This PR fixes the following warning:

archinstall/archinstall/lib/models/device.py:321:10 - error: Unnecessary isinstance call; "SectorSize" is always an instance of "SectorSize" (reportUnnecessaryIsInstance)

[codefiles]

I've thought about removing the following __post_init__ in the past.

archinstall/archinstall/lib/luks.py

Lines 43 to 45 in 390b495

	def __post_init__(self) -> None:
	if self.luks_dev_path is None:
	raise ValueError('Partition must have a path set')

Here is the luks_dev_path type annotation:

archinstall/archinstall/lib/luks.py

Lines 16 to 22 in 390b495

	@dataclass
	class Luks2:
	luks_dev_path: Path
	mapper_name: str | None = None
	password: Password | None = None
	key_file: Path | None = None
	auto_unmount: bool = False

It is checking for None instead of using isinstance so it does not trigger the reportUnnecessaryIsInstance Pyright warning. This is pretty much the same design defect though, an unnecessary type check in a post-init, right?

[correctmost]

This is pretty much the same design defect though, an unnecessary type check in a post-init, right?

Yeah :). I would like to enable mypy --strict-equality-for-none to catch these extraneous checks, but there are a bunch of places that need to be cleaned up first:

$ mypy --strict-equality-for-none
[...snip...]
archinstall/lib/luks.py:44: error: Non-overlapping identity check (left operand type: "Path", right operand type: "None")  [comparison-overlap]
[...snip...]

[codefiles]

Thanks!