Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Help keep advice in context #73099

Merged
merged 1 commit into from
May 11, 2021
Merged

Help keep advice in context #73099

merged 1 commit into from
May 11, 2021

Conversation

sbonds
Copy link
Contributor

@sbonds sbonds commented Mar 31, 2021

As part of an internal conversation around microsoft/PowerShell-DSC-for-Linux#764 it was suggested that we use python2 on RHEL8 since it was "recommended by Microsoft" in this document. This seems to be a misunderstanding of the context of this fix and goes against the strong preference for python3 in RHEL8. Help clarify that with this trivial improvement.

As part of an internal conversation around microsoft/PowerShell-DSC-for-Linux#764 it was suggested that we use python2 on RHEL8 since it was "recommended by Microsoft" in this document. This seems to be a misunderstanding of the context of this fix and goes against the strong preference for python3 in RHEL8. Help clarify that with this trivial improvement.
@PRMerger12
Copy link
Contributor

@sbonds : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@bwren
Copy link
Contributor

bwren commented May 10, 2021

#sign-off

@ktoliver ktoliver merged commit 1282fd0 into MicrosoftDocs:master May 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants