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

Fix type inference of parameters in classic functions #25172

Merged
merged 3 commits into from
Mar 18, 2025

Conversation

MartinGC94
Copy link
Contributor

PR Summary

Fixes type inference for parameters in classic function definitions where the parameters outside the scriptblock definition: function X ([string]$Param1){}

PR Context

I broke it with: #19830

PR Checklist

@iSazonov iSazonov added the CL-General Indicates that a PR should be marked as a general cmdlet change in the Change Log label Mar 14, 2025
…sitor.cs

Co-authored-by: Ilya <darpa@yandex.ru>
@iSazonov iSazonov self-assigned this Mar 18, 2025
@iSazonov

This comment was marked as outdated.

This comment was marked as outdated.

@iSazonov iSazonov merged commit 0ca17fd into PowerShell:master Mar 18, 2025
37 of 39 checks passed
Copy link
Contributor

microsoft-github-policy-service bot commented Mar 18, 2025

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

🔗 https://aka.ms/PSRepoFeedback

@MartinGC94 MartinGC94 deleted the FixParamInference branch March 18, 2025 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CL-General Indicates that a PR should be marked as a general cmdlet change in the Change Log
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants