Skip to content

fix: Abstract code knowledge of the notes function away from specific implementations#4761

Merged
rjsparks merged 3 commits into
ietf-tools:mainfrom
rjsparks:betternotes
Nov 15, 2022
Merged

fix: Abstract code knowledge of the notes function away from specific implementations#4761
rjsparks merged 3 commits into
ietf-tools:mainfrom
rjsparks:betternotes

Conversation

@rjsparks

Copy link
Copy Markdown
Member

No description provided.

@rjsparks rjsparks changed the title Betternotes fix: Abstract code knowledge of the notes function away from specific implementations Nov 15, 2022
@codecov

codecov Bot commented Nov 15, 2022

Copy link
Copy Markdown

Codecov Report

Merging #4761 (0bb6f5e) into main (cdfe7ea) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head 0bb6f5e differs from pull request most recent head 747653f. Consider uploading reports for the commit 747653f to get more accurate results

@@           Coverage Diff           @@
##             main    #4761   +/-   ##
=======================================
  Coverage   88.51%   88.51%           
=======================================
  Files         296      296           
  Lines       39845    39848    +3     
=======================================
+ Hits        35268    35272    +4     
+ Misses       4577     4576    -1     
Impacted Files Coverage Δ
ietf/meeting/views.py 90.59% <ø> (ø)
ietf/meeting/models.py 85.93% <100.00%> (+0.03%) ⬆️
ietf/doc/views_search.py 89.23% <0.00%> (ø)
ietf/doc/templatetags/ietf_filters.py 87.61% <0.00%> (+0.02%) ⬆️
ietf/nomcom/utils.py 91.54% <0.00%> (+0.24%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@rjsparks rjsparks merged commit 41eafa2 into ietf-tools:main Nov 15, 2022
@rjsparks rjsparks deleted the betternotes branch November 15, 2022 20:17
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Nov 19, 2022
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.

3 participants