Refactoring batch_format_link_properties to Utilize Optional Return Type for in-place Mutation

[mohamedawnallah]

Description

The current functionality within batch_format_link_properties doesn't utilize the returned links when the inplace parameter is set to True. It's more conventionally sound not to return a value if we're already mutating in place.

Changes Made

• Removed the return of links from batch_format_link_properties when the inplace parameter is True.
• Utilized Optional from the typing module to denote the possible absence of a return value when inplace is set to True.

How Has This Been Tested?

No alterations to the logic's behavior have been made. Testing ensured that the removal of the return value was correctly implemented, specifically when the inplace parameter is True.

Additional References

Python Optional typing hint:
https://docs.python.org/3/library/typing.html#typing.Optional

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.

[m-appel]

Thanks, I added one comment. In general we don't use type hints much in this project, I think I added this one out of habit.