Skip to content

fixes wrong variable used for property count - #380

Merged
vexx32 merged 1 commit into
PowerShellOrg:masterfrom
Maamue:master
Mar 20, 2020
Merged

vexx32 merged 1 commit into
PowerShellOrg:masterfrom
Maamue:master

Conversation

@Maamue

@Maamue Maamue commented Mar 20, 2020

Copy link
Copy Markdown
Contributor

PR Summary

This PR fixes a mismatched variable.
$ExpectedCount was assigned a value but $ExpectedPropertyCount was queried for the value.

Fixes ##367

Checklist

  • Pull Request has a meaningful title.
  • Summarised changes.
  • Pull Request is ready to merge & is not WIP.
  • [] Added tests / only testable interactively.
    • Make sure you add a new test if old tests do not effectively test the code changed.
  • Added documentation / opened issue to track adding documentation at a later date.

@vexx32 vexx32 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Thanks for fixing this one up! 😊 💖

@vexx32
vexx32 merged commit 497abaa into PowerShellOrg:master Mar 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants