Closed anubhavsinha98 closed 5 years ago
Merging #98 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #98 +/- ##
=======================================
Coverage 96.41% 96.41%
=======================================
Files 21 21
Lines 390 390
=======================================
Hits 376 376
Misses 14 14
Impacted Files | Coverage Δ | |
---|---|---|
app/core/domain/email_manager.py | 100% <ø> (ø) |
:arrow_up: |
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 1019c9c...59cef8a. Read the comment docs.
Hi @seanlip, below is the error message which was generated by the htmllint when we were using the social word.
Also we are using here htmllint
and I went through the wiki of htmllint
, and here it is mention :
https://github.com/htmllint/htmllint/wiki/Options#id-class-no-ad
Ah I see, thanks for explaining. Strange rule, but I think your fix is fine. Thanks!
This PR fixes the
htmllint
issue and fix the wrong-import-order.