feat: Use idnits3 for post submission nits check #8169
Conversation
| @@ -2996,6 +2996,24 @@ def test_raw_id(self): | |||
| self.should_succeed(dict(name=draft.name, rev='00',ext='txt')) | |||
| self.should_404(dict(name=draft.name, rev='00',ext='html')) | |||
|
|
|||
| @override_settings(IDNITS3_BASE_URL="https://example.org/idnits3") | |||
| def test_nits_url(self): | |||
| draft = WgDraftFactory(name="draft-ietf-moon-test",rev="01", create_revisions=range(0,2)) | |||
There was a problem hiding this comment.
A comment as to why you're setting up this particular arrangement of revisions would be helpful
|
If someone really wanted the v2 nits, can they get back to them quickly? |
|
Nudge:
|
No there are no quick links for that. Author tools just shows the output from idnits. |
|
so maybe instead of replacing the idnits2 button, we provide both idnits2 and idnits3 buttons until idnits3 becomes the thing in the submit path? |
|
Where can I find idnits3? |
@cabo, You can install with Footnotes |
One of the motivations of this change is to stop unnecessary idnits2 checks because this makes a call to datatracker which seems slow and could affect datatracker performance. |
OK, I have that, so "3.0.0-alpha.36" appears to be the most recent revision. https://author-tools.ietf.org/api/idnits3?url=https://www.ietf.org/archive/id/draft-ietf-cbor-cddl-more-control-07.xml |
Fixes #8107