8000 Cleanup unused pinvokes on unix by iSazonov · Pull Request #9531 · PowerShell/PowerShell · GitHub
[go: up one dir, main page]

Skip to content

Cleanup unused pinvoke 8000 s on unix#9531

Merged
iSazonov merged 5 commits intoPowerShell:masterfrom
iSazonov:cleanup-unused-pinvokes-on-unix
May 7, 2019
Merged

Cleanup unused pinvokes on unix#9531
iSazonov merged 5 commits intoPowerShell:masterfrom
iSazonov:cleanup-unused-pinvokes-on-unix

Conversation

@iSazonov
Copy link
Collaborator
@iSazonov iSazonov commented May 4, 2019

PR Summary

  • Remove unused TemporaryDirectory()
  • Remove unused GetUserName()
  • Use Environment.UserName on Unix
  • Use Environment.UserName and Environment.UserDomainName on Windows

PR Context

PR Checklist

@iSazonov iSazonov added the CL-CodeCleanup Indicates that a PR should be marked as a Code Cleanup change in the Change Log label May 4, 2019
@iSazonov iSazonov added this to the 7.0.0-preview.1 milestone May 4, 2019
@iSazonov iSazonov force-pushed the cleanup-unused-pinvokes-on-unix branch from 924c5a4 to b07bde8 Compare May 4, 2019 07:07
@iSazonov iSazonov force-pushed the cleanup-unused-pinvokes-on-unix branch from b0a8510 to a1488e8 Compare May 4, 2019 08:50
@iSazonov iSazonov self-assigned this May 4, 2019
@iSazonov iSazonov merged commit 2be3db8 into PowerShell:master May 7, 2019
@iSazonov iSazonov deleted the cleanup-unused-pinvokes-on-unix branch May 7, 2019 05:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-CodeCleanup Indicates that a PR should be marked as a Code Cleanup change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

0