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.

Sign up for GitHub

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

Jump to bottom

doc: note that tests should include a description #9415

Merged
merged 2 commits into from
Nov 5, 2016
Merged

doc: note that tests should include a description #9415

merged 2 commits into from
Nov 5, 2016

Commits on Nov 5, 2016

  1. doc: note that tests should include a description 

    Update the Writing Tests guide to specify that tests should include a
brief description of what they are designed to test.

PR-URL: nodejs#9415

Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>oc
    @gibfahn
    gibfahn committed Nov 5, 2016
    Configuration menu
    Copy the full SHA
    3e6cc60 View commit details
    Browse the repository at this point in the history

  2. test: update http-header-obstext 

    PR-URL: nodejs#9415

Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
    @gibfahn
    gibfahn committed Nov 5, 2016
    Configuration menu
    Copy the full SHA
    a99b441 View commit details
    Browse the repository at this point in the history