Skip to content

fix: Use String.prototype.localeCompare() when sorting table rows#6441

Merged
rjsparks merged 1 commit intoietf-tools:mainfrom
larseggert:fix-6437
Oct 10, 2023
Merged

fix: Use String.prototype.localeCompare() when sorting table rows#6441
rjsparks merged 1 commit intoietf-tools:mainfrom
larseggert:fix-6437

Conversation

@larseggert
Copy link
Copy Markdown
Collaborator

Also fix some js warnings and indentation nits while I'm here.

Fixes #6437

@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 6, 2023

Codecov Report

Merging #6441 (4420b58) into main (c5a955d) will decrease coverage by 0.12%.
Report is 27 commits behind head on main.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main    #6441      +/-   ##
==========================================
- Coverage   88.81%   88.69%   -0.12%     
==========================================
  Files         290      290              
  Lines       40879    40431     -448     
==========================================
- Hits        36305    35859     -446     
+ Misses       4574     4572       -2     
Files Coverage Δ
ietf/doc/views_statement.py 94.96% <ø> (ø)
ietf/nomcom/urls.py 100.00% <ø> (ø)
ietf/nomcom/utils.py 91.42% <ø> (-2.01%) ⬇️
ietf/nomcom/views.py 93.00% <ø> (-1.62%) ⬇️
ietf/ipr/forms.py 87.98% <66.66%> (-0.28%) ⬇️

... and 2 files with indirect coverage changes

@rjsparks rjsparks merged commit a02812b into ietf-tools:main Oct 10, 2023
@larseggert larseggert deleted the fix-6437 branch October 10, 2023 15:51
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Oct 14, 2023
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.

Wrong collation sorting in iesg/discusses/

3 participants