Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Use Environment.ProcessId in SpecialVariables.PID#24926

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
iSazonov merged 3 commits intoPowerShell:masterfromfMichaleczek:CA1837
Feb 4, 2025

Conversation

fMichaleczek
Copy link
Contributor

@fMichaleczekfMichaleczek commentedFeb 3, 2025
edited
Loading

PR Summary

Continue#14063

Applying CA1837https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca1837

PR Context

ImproveMicrosoft.PowerShell.SDK doesn't work in a AspNetCore Blazor WebAssembly project

Process.GetCurrentProcess().Id is not available on net9.0-Browser.

With Environment.ProcessId, the API doesn't throw anymore :

image

PR Checklist

@iSazonov

This comment was marked as outdated.

@azure-pipelinesAzure Pipelines

This comment was marked as outdated.

@iSazonov

This comment was marked as outdated.

@iSazonov

This comment was marked as outdated.

@azure-pipelinesAzure Pipelines

This comment was marked as outdated.

@iSazonoviSazonov added the CL-CodeCleanupIndicates that a PR should be marked as a Code Cleanup change in the Change Log labelFeb 4, 2025
@iSazonov

This comment was marked as outdated.

@azure-pipelinesAzure Pipelines

This comment was marked as outdated.

@iSazonoviSazonov self-assigned thisFeb 4, 2025
@iSazonoviSazonov changed the titleContinue Enable CA1837: Use 'Environment.ProcessId'Use Environment.ProcessId in SpecialVariables.PIDFeb 4, 2025
@iSazonoviSazonov merged commit141da81 intoPowerShell:masterFeb 4, 2025
39 of 41 checks passed
@microsoft-github-policy-serviceMicrosoft GitHub Policy Service
Copy link
Contributor

microsoft-github-policy-servicebot commentedFeb 4, 2025
edited by unfurl-linksbot
Loading

📣 Hey@fMichaleczek, how did we do? We would love to hear your feedback with the link below! 🗣️

🔗https://aka.ms/PSRepoFeedback

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@iSazonoviSazonoviSazonov approved these changes

Assignees

@iSazonoviSazonov

Labels
CL-CodeCleanupIndicates that a PR should be marked as a Code Cleanup change in the Change Log
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@fMichaleczek@iSazonov@TravisEz13

[8]ページ先頭

©2009-2025 Movatter.jp