Skip to content

fix: count RFCs correctly for doc/ad#6833

Merged
rjsparks merged 2 commits intoietf-tools:mainfrom
rjsparks:adstatecounts
Jan 3, 2024
Merged

fix: count RFCs correctly for doc/ad#6833
rjsparks merged 2 commits intoietf-tools:mainfrom
rjsparks:adstatecounts

Conversation

@rjsparks
Copy link
Copy Markdown
Member

Fixes #6814

This whole function needs to be refactored and made testable.

@rjsparks rjsparks requested review from jennifer-richards and larseggert and removed request for jennifer-richards December 21, 2023 21:15
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 21, 2023

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (f3a574c) 88.78% compared to head (604b298) 88.79%.
Report is 5 commits behind head on main.

Files Patch % Lines
ietf/doc/views_search.py 66.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6833      +/-   ##
==========================================
+ Coverage   88.78%   88.79%   +0.01%     
==========================================
  Files         285      285              
  Lines       40324    40370      +46     
==========================================
+ Hits        35800    35845      +45     
- Misses       4524     4525       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rjsparks rjsparks merged commit bbc64d3 into ietf-tools:main Jan 3, 2024
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jan 7, 2024
@rjsparks rjsparks deleted the adstatecounts branch January 9, 2024 21:13
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.

RFC column empty

2 participants