dfinke/PSharp

ConvertTo-CamelCase

Opened this issue · 5 comments

toenuff commented

There's a bug in ConvertTo-CamelCase where only the first cmdlet is properly converted. Other cmdlets in the string get converted, but they lose the hyphen.

Please no one fix this. I have coded the fix and will be giving a live demo of the pull request during Techstravaganza tomorrow.

dfinke commented

Cool. Looking forward to hearing how the demo goes. I'll merge the change after.

toenuff commented

Well the demo went poorly for this part :P I just realized why. I showed off my fancy pester tests, but I forgot that the test might need the ISE to run :P

dfinke commented

Ouch, bummer.

Sounds like in need to add a $Host.Name guard.

toenuff commented

Well, it's there now :) I have til the PS deep dive to fix my github demos - I'll just find something else to fix of yours (that doesn't require ISE) - should be easy - finding code of yours with bugs, that is :P

dfinke commented

Thanks for the pull request, I'll merge it.