Templating: don't check the users who posted reviews#256
Templating: don't check the users who posted reviews#256trexfeathers wants to merge 3 commits intoSciTools:mainfrom
Conversation
HGWright
left a comment
There was a problem hiding this comment.
Would it be possible to have some hidden text, so that we can go from "unlikely" that someone accidentally posts a review like that, to "near impossible" that someone could accidentally match the format this check looks for?
Good idea - 28d988e. I've edited this comment to conform, as an easy way to test that this works: SciTools/python-stratify#404 (review) After merging this PR: if we re-run the CI on SciTools/python-stratify#404 we should see the "test" disappear from the end when the automation overwrites the comment. |
gh api useris not available when running as part of CI. Plus some information statesscitools-ciand other information statesscitools-ci[bot], so it's difficult to program this reliably without a lot of testing.In reality: any review which begins with the Templating level 2 heading is likely to be the appropriate one to modify, so I propose not bothering to check the user at all.