Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Fix some HTML validation errors that came in via #4035 #4066

Merged
merged 3 commits into from
Jun 16, 2022

Conversation

larseggert
Copy link
Collaborator

Also make validation errors display a bit better.

Also make validation errors display a bit better.
@codecov
Copy link

codecov bot commented Jun 8, 2022

Codecov Report

Merging #4066 (cbcbb76) into main (8b6dbf3) will decrease coverage by 0.00%.
The diff coverage is n/a.

❗ Current head cbcbb76 differs from pull request most recent head 72c2947. Consider uploading reports for the commit 72c2947 to get more accurate results

@@            Coverage Diff             @@
##             main    #4066      +/-   ##
==========================================
- Coverage   88.14%   88.14%   -0.01%     
==========================================
  Files         297      297              
  Lines       39275    39275              
==========================================
- Hits        34620    34618       -2     
- Misses       4655     4657       +2     
Impacted Files Coverage Δ
ietf/urls.py 81.08% <ø> (ø)
ietf/utils/pipe.py 82.60% <0.00%> (-4.35%) ⬇️
ietf/submit/forms.py 82.92% <0.00%> (-0.23%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7fc601e...72c2947. Read the comment docs.

ietf/templates/base/icons.html Outdated Show resolved Hide resolved
@rjsparks rjsparks merged commit 411b5f4 into ietf-tools:main Jun 16, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 19, 2022
@larseggert larseggert deleted the fix-html-validation-nits branch July 5, 2022 18:47
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