-
Notifications
You must be signed in to change notification settings - Fork 9
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 file system casing #33
base: main
Are you sure you want to change the base?
Conversation
Sorry @gaelcolas , but the build task keeps failing. I've added PlatyPS after the first failed build, now it fails with a different error. The change I did was just file path casing ;) |
I will look into this. It should not run two of the tasks if PlatyPS is not available. I will let you know when you can revert the change to required modules. |
Okay, a fix has now been released in DocGenerator so if PlatyPS is removed from the RequiredModules.ps1 this PR should pass build. |
This reverts commit bdc855b.
We have to wait for @gaelcolas to merge this as I don't have permission in this repo. 🙂 |
@gaelcolas, can we merge this AFTER having done a full release? |
Pull Request (PR) description
Using proper capitalization as used in the module folder structure to enable Linux usage as well. Tested with the DscWorkshop project while attempting to build Machine Configuration packages :)
This Pull Request (PR) fixes the following issues
Task list
Entry should say what was changed and how that affects users (if applicable), and
reference the issue being resolved (if applicable).
help.
This change is