Conversation
|
@JustinGrote - thank you for this pr. Hey Justin - @bhavanakonchada and I are reviewing the change - Justin, can you change the conditional to the following logic? If lowercase only exists - do nothing |
Adding to what @theJasonHelmick said, we need this change to avoid any disruption for existing customers as a lot could be riding on this path. |
|
@bhavanakonchada @theJasonHelmick I added migration logic as appropriate and tried to handle all possible edge cases, I did not do this transparently as users should be informed what is going on. |
|
Thank you @JustinGrote - much appreciated. Will review with @bhavanakonchada with the plan to accept the PR after testing. |
I've delayed the review and merge of the PR to October 2023 post feature release for Ephemeral and UX2. Thank you for this fix - looking forward to have it included soon. WE may need to adjust the code for our own purposes, but the credit goes to you @JustinGrote for the catch and code. Thank you! |
The powershell profile folders have incorrect casing (PowerShell instead of powershell) to the powershell default, and cause problems with scripts that interact with them. This PR fixes the casing.
Related Discussion: #302