You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Spelling fixes in our doc
Found using `mdspell -a -n --en-us "**/*.md"`. At some point we might want to have a built in ignore dictionary to avoid false positives.
(cherry picked from commit 7b87070)
Resolved result ref order can affect unit test outcome
Test TestTaskParamResolver_ResolveResultRefs performs a diff on a slice
of resolved result references. Unfortunately the ordering of this slice
is not guaranteed so sometimes the test fails.
This PR updates the unit test to check each entry appears as expected in
the tt.want slice (confirming also that the lengths of the slices
match). Also adds a small helper func to format the resolved result refs
for printing in the event that they don't match.
Set scm.PR.Sha to scm.PR.Head.Sha when loading from disk.
This guarantee is made when retrieving a PR from a provider, but we do not persist
that guarantee when roundtripping from disk.