PSGraph icon indicating copy to clipboard operation
PSGraph copied to clipboard

Replace Invoke-Expression with Invoke-Item

Open danhje opened this issue 5 years ago • 3 comments

Invoke-Expression gave error on certain versions of PowerShell. Invoke-Item is a better way of opening a file using the default program or action.

danhje avatar Feb 03 '20 09:02 danhje

Thank you for your patience, not sure how I missed this one.

Good call on using Invoke-Item instead. Code looks good as it is, I just need to test it. I don't think I have any tests for that line, so I may add one with a mock before merging.

KevinMarquette avatar Sep 16 '20 18:09 KevinMarquette

Hey, @KevinMarquette!

Have you had time to take a look at this?

moiaune avatar May 31 '21 17:05 moiaune

Hi @KevinMarquette

I think it would be a good change. I am writing as a reminder. If you would be so kind as to spend some time on it, we would appreciate it.

🙏Thank you for making this very useful module available.

Best regards Claudio

claudio-salvio avatar Mar 18 '22 21:03 claudio-salvio