Samuel D. Leslie

Hey there, Thanks for the above! I'll get back to your regarding the SCPs but with respect to Azure AD Connect: The referenced permission (`DS-Replication-Get-Changes-All`) is technically only required for...

Nope, because in a custom configuration you can create the AD connector account yourself, then use the `ADSyncConfig` PowerShell module included with AADC to set the appropriate permissions based on...

Yes, but technically that's not required either as the `ms-DS-ConsistencyGuid` attribute doesn't have to be used as the source anchor. It's a kludge, but you could perhaps look for the...

Regarding the SCPs, I don't think filtering on the `serviceClassName` attribute alone makes sense. For the `serviceConnectionPoint` class it's [documented](https://docs.microsoft.com/en-us/windows/win32/adschema/c-serviceconnectionpoint) as an optional attribute. The only mandatory attribute for the...

Good point, and glad it's shown, but I'd suggest there should still be an entry in the SCP section which reflects the fact that hybrid Azure AD Join is enabled...

I've submitted a PR which adds this functionality. See PR #102 for more details.

@texastoland Spawning `shfmt` with the current working directory set to that of the file being saved was the approach I originally took, but it's unfortunately not sufficient. As noted in...

@texastoland I don't think I understand the approach you're suggesting?

The thing is we're already inconsistent with the command line. The changes in this PR bring the behaviour of the extension in-line with the results from running `shfmt` from the...

Great work, thanks @kddejong.