Skip to content

feat: updated boostrap5 work - #3596

Merged
NGPixel merged 116 commits into
ietf-tools:feat/bs5from
painless-security:personal/jennifer/7.45.1.dev0.bootstrap-merge
Mar 8, 2022
Merged

feat: updated boostrap5 work#3596
NGPixel merged 116 commits into
ietf-tools:feat/bs5from
painless-security:personal/jennifer/7.45.1.dev0.bootstrap-merge

Conversation

@jennifer-richards

Copy link
Copy Markdown
Member

Tests pass except for the code coverage, which dropped slightly.

======================================================================
FAIL: code_coverage_test (ietf.utils.test_runner.CoverageTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/root/src/ietf/utils/test_runner.py", line 392, in code_coverage_test
    checker.stop()
  File "/root/src/ietf/utils/test_runner.py", line 332, in report_test_result
    ( test, test_coverage*100, latest_coverage_version, master_coverage*100, ))
AssertionError: 0.8805695792880259 not greater than or equal to 0.8809691174563656 : The code coverage percentage is now lower (88.06%) than for version 7.46.0 (88.12%)

----------------------------------------------------------------------
Ran 1344 tests in 1037.548s

FAILED (failures=1, skipped=2)

 - Legacy-Id: 19573
 - Legacy-Id: 19577
 - Legacy-Id: 19580
 - Legacy-Id: 19582
 - Legacy-Id: 19584
Only load the module, no other changes yet.
 - Legacy-Id: 19586
https://datatables.net/ has way more features and better support, and bootstrap5
styling.
 - Legacy-Id: 19587
 - Legacy-Id: 19588
 - Legacy-Id: 19589
 - Legacy-Id: 19590
 - Legacy-Id: 19591
 - Legacy-Id: 19593
 - Legacy-Id: 19594
 - Legacy-Id: 19606
 - Legacy-Id: 19611
 - Legacy-Id: 19612
 - Legacy-Id: 19613
 - Legacy-Id: 19615
 - Legacy-Id: 19616
 - Legacy-Id: 19617
 - Legacy-Id: 19618
larseggert and others added 27 commits February 8, 2022 17:29
 - Legacy-Id: 19913
 - Legacy-Id: 19914
and fix/suppress errors found by that.
 - Legacy-Id: 19916
…t reconciling differences.

 - Legacy-Id: 19945
 - Legacy-Id: 19978
 - Legacy-Id: 19980
 - Legacy-Id: 19982
Hopefully fix ietf-tools#3556.
 - Legacy-Id: 19983
Note: SVN reference [19978] has been migrated to Git commit ca167c0
Make the js a bit less noisy and adjust the tests accordingly.
 - Legacy-Id: 19984
Note: SVN reference [19979] has been migrated to Git commit f61ddb4
Fix # 3555.
 - Legacy-Id: 19985
Note: SVN reference [19980] has been migrated to Git commit 686f835
Properly URL-encode and -decode a bunch of query parameters. [from revision 19,981]
 - Legacy-Id: 19986
…5.1.dev0.bootstrap-merge

# Conflicts:
#	ietf/templates/meeting/session_buttons_include.html
…5.1.dev0.bootstrap-merge

# Conflicts:
#	bootstrap/package.json
#	ietf/templates/meeting/session_buttons_include.html
Only add right-hand page navigation panel to #content elements that have
the ietf-auto-nav class and test that the base template's DOM is compatible
with the panel.
…5.1.dev0.bootstrap-merge

# Conflicts:
#	bootstrap/package-lock.json
#	bootstrap/package.json
#	package-lock.json
#	package.json
@NGPixel
NGPixel merged commit 4ef9f5c into ietf-tools:feat/bs5 Mar 8, 2022
@jennifer-richards
jennifer-richards deleted the personal/jennifer/7.45.1.dev0.bootstrap-merge branch March 9, 2022 16:58
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jun 16, 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