Repository navigation
Fix logging in parallel runspaces - #7
Conversation
+semver:feature
+semver:feature
I think this will still work for PS2, but I'm not 100% sure!
159e907 to
8335c04
Compare
|
@dlwyatt this last commit changes the build.ps1 script I added to accept a Having done that, I can also update all the version numbers in the build output (in the library file and in the metadata), so I went ahead and did all of that in the build script too. I stuck it in a single commit, so if you don't want it, I can revert that commit ... |
Previously, running the runspace or invoke tests would break each other (and the simple) test. +semver:feature
ParallelRunspace tests (ThreadJobs) are WIP: currently fail
09444c9 to
3a6fd10
Compare
However, his last commit demonstrates a remaining problem found in our QA: Parallel runspaces in the same process still cause problems due to the Personally I'd like to see this merged, and will create a new issue for the other problem... |
| } else { | ||
| Write-Warning "Skipping PowerShell Core tests" | ||
| } | ||
| } No newline at end of file |
|
While setting up an automated build against my branch I ran into some issues with the tests ... |
|
FWIW, these changes passed our tests and QA and we're now working with a forked version ... |
|
Cool. I’m on vacation for a couple of weeks. I can merge the PR now with my phone, but won’t be able to publish until I get back home |
This fixes #6 and also fixes the module to work in PowerShell 6 and 7