And also fix a couple of bugs found when writing tests for the matching used
by tamarack bot. This expands the files that the core team will be requested for
on pull request reviews.
`team-windows` should be requested for reviews whenever a windows test file
or the reg.py execution module and accompanying reg.py state file are edited.
The CODEOWNERS file documentation for GitHub says they use a .gitignore-style
file matching pattern. However, since [tamarack](https://github.com/rallytime/tamarack)
is doing the actual team review requesting work, it uses Python's fnmatch lib.
This PR updates some of the file matching patterns to use fnmatch more specifically
and will improve the review requests to be more accurate and increase the number
of automatic team review requests.
This ensures the file displays correctly on GitHub. This also fixes a
few minor typos and the format of the Vim modeline, which wasn't being
detected previously.
The most "important" change here is to remove the checkboxes for
"Tests Written" on the pull request template. Because they're checkboxes
rather than radio buttons, no matter which one you checked it would show
"1/2 complete" for any references to that PR in other issues.
