Closed jkaufman-mitre closed 1 month ago
This feels like it has expanded FAR beyond what is outlined in the title. Can we update the title of the PR and description to inform what we should be looking for here?
This feels like it has expanded FAR beyond what is outlined in the title. Can we update the title of the PR and description to inform what we should be looking for here?
I think what we're seeing is due to all the baseline changes that have been merged to main after this PR was made. Honestly, if that's what's going on, since there so many changes have been made since this PR was submitted, it would probably be easiest to close this PR and open a new one, rather than try to resolve all the merge conflicts.
EDIT: I tried changing to target branch from main then back to main, because sometimes that helps clean up the presentation on GitHub, but it didn't help this time.
Going to close and reopen to try and fix as mentioned above
๐ฃ Description
Changed the max number of Super Admin accounts from 4 to 8 in order to be consistent with the Microsoft baseline policy.
Fixes #252
๐ญ Motivation and context
Comment from SNL recommending to be consistent with Microsoft.
๐งช Testing
โ Pre-approval checklist
โ Pre-merge Checklist
Squash and merge
button.โ Post-merge Checklist