Skip to content

feat: require draft revisions in ipr disclosures. Explain missing ones. - #6160

Merged
rjsparks merged 5 commits into
ietf-tools:mainfrom
rjsparks:ipr_require_revisions
Aug 18, 2023
Merged

feat: require draft revisions in ipr disclosures. Explain missing ones.#6160
rjsparks merged 5 commits into
ietf-tools:mainfrom
rjsparks:ipr_require_revisions

Conversation

@rjsparks

@rjsparks rjsparks commented Aug 16, 2023

Copy link
Copy Markdown
Member

Fixes #6155

@codecov

codecov Bot commented Aug 16, 2023

Copy link
Copy Markdown

Codecov Report

Merging #6160 (97a2e47) into main (f2b716f) will increase coverage by 0.00%.
Report is 2 commits behind head on main.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #6160   +/-   ##
=======================================
  Coverage   88.68%   88.69%           
=======================================
  Files         290      290           
  Lines       40264    40300   +36     
=======================================
+ Hits        35708    35743   +35     
- Misses       4556     4557    +1     
Files Changed Coverage Δ
ietf/ipr/forms.py 88.25% <100.00%> (+0.34%) ⬆️
ietf/ipr/templatetags/ipr_filters.py 85.71% <100.00%> (+28.57%) ⬆️

... and 2 files with indirect coverage changes

@jennifer-richards jennifer-richards left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved! (A couple style nits and a possible (but not required) refactor noted inline.)

Comment thread ietf/ipr/forms.py Outdated
Comment thread ietf/ipr/templatetags/ipr_filters.py Outdated
Comment thread ietf/ipr/templatetags/ipr_filters.py Outdated

@larseggert larseggert left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

draft -> Internet-Draft everywhere

@rjsparks

Copy link
Copy Markdown
Member Author

This also needs to test another facet - commit incoming

@rjsparks
rjsparks merged commit 53be2c3 into ietf-tools:main Aug 18, 2023
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 22, 2023
@rjsparks
rjsparks deleted the ipr_require_revisions branch October 2, 2023 17:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Require revisions on draft specific IPR disclosures

3 participants