pdksync - (maint) - Updating Contribution Guidelines

…ows_2022_support

pdksync - (FM-8922) - Add Support for Windows 2022

Prior to this commit, there was an attempt to fix an issue where puppet
could not handle a trailing '/' at the end of a physicalpath. While it
stopped users from creating resources with trailing '/', it did not fix
existing IIS entities that already had the trailing '/'.

This commit aims to complement the previous code by removing trailing
'/' in physicalpaths for existing resources.

Prior to this commit there were some unit tests that would fail due to
the additional checks inserted in the latest fix. Some of the previous
code would also not work as intended and lint was throwing warnings at
some formatting choices made.

This commit aims to provide a number of small fixes to deal with both,
lint warnings and small errors in previous code that would make unit
tests fail.

Prior to this commit several fixes were implemented to deal with the
problem described in the JIRA ticket MODULES-11188. As a result of those
changes, the behaviour of the module regarding physicalpaths was
slighly altered.

This commit aims to update the README file to inform users about the
previously existing issue and the measures that the module will take to
deal with it.

MODULES-11188: trim physicalpath for iis_application resource

Prior to this commit it looks like we'd chosen the incorrect property
to pull back from the iis config. PhysicalPath != physicalPath and the
former was behaving differently between the app and site resources.

This commit standardises on physicalPath which returns the correct value
for both app and site resources. For clarity, physicalPath returns a
string which we can directly manipulate if required.

…ixes

(MODULES-11188) Fix physicalPath on apps and sites

Release prep v8.1.0