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

Add linting information and test failures to html #448

Merged
merged 1 commit into from
Sep 21, 2015

Conversation

Marsup
Copy link
Contributor

@Marsup Marsup commented Sep 21, 2015

First, sorry for the massive refactor ;)

This patch :

  1. adds tooltips on most elements of the page (Document html coverage report source markup #439)
  2. adds margin marks in the coverage section about linting errors
  3. repeats those linting errors at the bottom of the report in case coverage is not enabled
  4. adds a list of test script errors at the bottom of test errors

[1]
1
2
[2]
3
[3]
4
[4]
5

@geek geek added the feature New functionality or improvement label Sep 21, 2015
@geek
Copy link
Member

geek commented Sep 21, 2015

@Marsup you are on a roll... thank you!

@geek geek self-assigned this Sep 21, 2015
@geek geek added this to the 5.17.1 milestone Sep 21, 2015
geek added a commit that referenced this pull request Sep 21, 2015
Add linting information and test failures to html
@geek geek merged commit 4bcdb49 into hapijs:master Sep 21, 2015
@Marsup
Copy link
Contributor Author

Marsup commented Sep 21, 2015

Consequences of giving a nodejs training last week and seeing trainees lost as to why the tests failed while the report shown nothing ;)

@skeggse
Copy link
Contributor

skeggse commented Sep 22, 2015

Marvelous, thanks!

@Marsup Marsup deleted the html-lint branch December 18, 2015 19:25
@lock
Copy link

lock bot commented Jan 9, 2020

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.

@lock lock bot locked as resolved and limited conversation to collaborators Jan 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature New functionality or improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants