Skip to content

Conversation

PolonaM
Copy link
Collaborator

@PolonaM PolonaM commented Feb 6, 2023

Type annotation added to user and user_info module

@PolonaM PolonaM marked this pull request as draft February 6, 2023 12:19
@PolonaM PolonaM force-pushed the US01_type_annotation branch 3 times, most recently from d188d9f to 6d0a380 Compare February 7, 2023 12:38
@PolonaM PolonaM marked this pull request as ready for review February 7, 2023 13:46
@@ -0,0 +1,3 @@
---
minor_changes:
- Added type annotation to user modules. (https://github.com/ScaleComputing/HyperCoreAnsibleCollection/pull/96)
Copy link
Collaborator

@domendobnikar domendobnikar Feb 7, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is probably not needed.
Document fragments should be used in case of an "operational/functional" change.
Examples: Bugfix, adding/removing/altering input or output, new module, new functionality. - Something that impacts the final user.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree, I removed the fragment

Copy link
Collaborator

@domendobnikar domendobnikar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 👍

@PolonaM PolonaM force-pushed the US01_type_annotation branch from a4e2f44 to 29a00f1 Compare February 8, 2023 10:55
@PolonaM PolonaM merged commit 54ce748 into main Feb 8, 2023
@domendobnikar domendobnikar deleted the US01_type_annotation branch February 8, 2023 13:03
@domendobnikar domendobnikar restored the US01_type_annotation branch February 8, 2023 13:03
@domendobnikar domendobnikar deleted the US01_type_annotation branch February 8, 2023 13:03
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.

2 participants