chore: add type annotation to cloudinit.config.cc_growpart

[JaneSoo]

ref: #5445 (comment)
chore: add type annotation to cloudinit.config.cc_growpart

• Added type annotations to all functions and methods
  • fs as Optional[str]: Can be None based on filesystem detection
  • info will be List[Tuple[str, str, str]] of device name, action/status and message. Eg: ("/dev/sda1", "CHANGED", "changed (/dev/sda, 1) from 1024 to 2048")
• Removed cloudinit.config.cc_growpart from mypy exclusion list

[holmanb]

This defeats the purpose of type annotations. You label it as Optional, but then throw an exception on the first line - this means that a caller that passes a NoneType will not by warned by type checking that this will crash the program.

[JaneSoo]

thanks for reviewing, I've removed the assert part and make them required str.

[holmanb]

Without dropping the module from the mypy overrides, this doesn't enable type checking on the module.

[JaneSoo]

Without dropping the module from the mypy overrides, this doesn't enable type checking on the module.

@holmanb I think the module was dropped in this commit 3082dcb#diff-50c86b7ed8ac2cf95bd48334961bf0530cdc77b5a56f852c5c61b89d735fd711L39

[github-actions]

Hello! Thank you for this proposed change to cloud-init. This pull request is now marked as stale as it has not seen any activity in 14 days. If no activity occurs within the next 7 days, this pull request will automatically close.

If you are waiting for code review and you are seeing this message, apologies! Please reply, tagging blackboxsw, and he will ensure that someone takes a look soon.

(If the pull request is closed and you would like to continue working on it, please do tag blackboxsw to reopen it.)