Closed JPRuskin closed 1 month ago
Looks like I'm going to need to set some secrets before this can merge! Didn't expect it to try and run the actions -interesting.
Ah, but I don't have permissions. May need Stevie, here?
104 tests 104 :white_check_mark: 23s :stopwatch: 14 suites 0 :zzz: 4 files 0 :x:
Results for commit 9b1cd877.
:recycle: This comment has been updated with latest results.
This seems fine!
Description Of Changes
Adds a new workflow for testing Quickstart Guide before merging any pull requests.
Please go and have a look at the result here and see if there's anything you'd like to change. This PR is in draft until folk have had a chance to have a look, at which point I'll squash things and fixup the commit history a little.
Expectations:
Motivation and Context
We want to provide more automated testing, to ensure issues are caught before things are merged in.
Big Change Notes
For those reading this who frequently (or less frequently) use the Quickstart Guide, here is a list of the biggest scary changes:
TrustedPeople
. There should no longer need to be a copy inMy
.-SkipBrowserLaunch
added to Set-SSL. That will automatically happen when the user is not in ConsoleHost, too (e.g. remoting or in ISE).Testing
Change Types Made
Change Checklist
Related Issue
Fixes #253