A PR approval checker got stricter about which words may appear under method keys.
New method-key validation rules exist on the PR approve validator; whether it runs in normal use could not be resolved.
What's wrong with this entry?
The in-page PR approve binding validator now inspects method keys and refuses values that do not belong under them.
- On the approve stamp, only a value matching /^create$/i may ride a method key.
- On the paired freshness read, only a value matching /^get$/i may.
- Each violation is rejected with its own message, and the create word under a non-method key is also refused.
- Whether this validator runs in normal use could not be resolved from the build.
is a method key holding a non-method value
Strings lifted out of the shipped bundle, so the claim above can be checked against them.
Related
Other releases about the same thing. Found by shared names or similar wording; neither means one caused the other.
-
v2.1.219
PR-review anchor repo names reject "." and ".."
Both mention code review
-
v2.1.219
Composed PR-review publish refusal explains whether it is a live switch
Both mention code review
-
v2.1.239
Code review findings now carry a short summary and a category
Both mention code review