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

test: remove unused modules #4684

Closed
wants to merge 15 commits into from
Closed

Commits on Jan 13, 2016

  1. tools: implement no-unused-vars for eslint

    PR-URL: nodejs#4536
    Reviewed-By: Rod Vagg <[email protected]>
    Reviewed-By: Michaël Zasso <[email protected]>
    Reviewed-By: Colin Ihrig <[email protected]>
    Trott authored and Myles Borins committed Jan 13, 2016
    Configuration menu
    Copy the full SHA
    31b6580 View commit details
    Browse the repository at this point in the history
  2. doc: fix numbering in stream.markdown

    PR-URL: nodejs#4538
    Reviewed-By: Colin Ihrig <[email protected]>
    richardsun29 authored and Myles Borins committed Jan 13, 2016
    Configuration menu
    Copy the full SHA
    54e4c09 View commit details
    Browse the repository at this point in the history
  3. doc: bring releases.md up to date

    Refs: nodejs#3254
    PR-URL: nodejs#4540
    Reviewed-By: James M Snell <[email protected]>
    cjihrig authored and Myles Borins committed Jan 13, 2016
    Configuration menu
    Copy the full SHA
    5d6e1c2 View commit details
    Browse the repository at this point in the history
  4. doc: add missing backtick for readline

    PR-URL: nodejs#4549
    Reviewed-By: Colin Ihrig <[email protected]>
    mscdex authored and Myles Borins committed Jan 13, 2016
    Configuration menu
    Copy the full SHA
    8cb8840 View commit details
    Browse the repository at this point in the history
  5. doc: add Myles Borins to Release Team

    Adds Myles Borins and his public key to the README
    
    PR-URL: nodejs#4578
    Reviewed-By: Colin Ihrig <[email protected]>
    Reviewed-By: Evan Lucas <[email protected]>
    Myles Borins committed Jan 13, 2016
    Configuration menu
    Copy the full SHA
    719b145 View commit details
    Browse the repository at this point in the history
  6. doc: add Evan Lucas to Release Team

    Adds Evan Lucas and his public key to the README for releases
    
    PR-URL: nodejs#4579
    Reviewed-By: Colin Ihrig <[email protected]>
    Reviewed-By: Myles Borins <[email protected]>
    evanlucas authored and Myles Borins committed Jan 13, 2016
    Configuration menu
    Copy the full SHA
    a28e51a View commit details
    Browse the repository at this point in the history
  7. doc: fix description about the latest-codename

    fix description about the latest LTS release download page
    to make it clear
    
    PR-URL: nodejs#4583
    Reviewed-By: Rod Vagg <[email protected]>
    Reviewed-By: Colin Ihrig <[email protected]>
    Reviewed-By: James M Snell <[email protected]>
    JungMinu authored and Myles Borins committed Jan 13, 2016
    Configuration menu
    Copy the full SHA
    5143719 View commit details
    Browse the repository at this point in the history
  8. doc: label http.IncomingMessage as a Class

    `Server`, `ServerResponse` etc. were marked as classes, this one class
    was overlooked.
    
    PR-URL: nodejs#4589
    Fixes: nodejs#4576
    Reviewed-By: James M Snell <[email protected]>
    Reviewed-By: Colin Ihrig <[email protected]>
    Reviewed-By: Roman Reiss <[email protected]>
    Sequoia authored and Myles Borins committed Jan 13, 2016
    Configuration menu
    Copy the full SHA
    367a13d View commit details
    Browse the repository at this point in the history
  9. doc: document http's server.listen return value

    The server method returns `self` in order to allow chaining.
    
    PR-URL: nodejs#4590
    Fixes: nodejs#4571
    Reviewed-By: James M Snell <[email protected]>
    Reviewed-By: Roman Reiss <[email protected]>
    Sequoia authored and Myles Borins committed Jan 13, 2016
    Configuration menu
    Copy the full SHA
    80ad952 View commit details
    Browse the repository at this point in the history
  10. doc: adds usage of readline line-by-line parsing

    In order to make developers aware of node-core built-in
    functionality, which might replace module APIs, we should
    add an example of readline`s interface usage.
    SEO will eventually aid this goal, since it is well searched
    on Q&A sites.
    
    PR-URL: nodejs#4609
    Reviewed-By: Roman Reiss <[email protected]>
    Reviewed-By: James M Snell <[email protected]>>
    eljefedelrodeodeljefe authored and Myles Borins committed Jan 13, 2016
    Configuration menu
    Copy the full SHA
    d9768a4 View commit details
    Browse the repository at this point in the history
  11. doc: update stylesheet to match frontpage

    - Changed colors to match frontpage as close as possible.
    - Links are slightly more horizontally padded as compared before to
      accomodate for the hover effect.
    - Slightly reduced the scroll indication height on the TOC.
    - The main content is now offset using margin instead of the previous
      border hack.
    - remove empty footer that was rendering a dark bar on the bottom of
      each page without any content.
    
    PR-URL: nodejs#4621
    Reviewed-By: Jeremiah Senkpiel <[email protected]>
    Reviewed-By: James M Snell <[email protected]>
    Reviewed-By: Brian White <[email protected]>
    silverwind authored and Myles Borins committed Jan 13, 2016
    Configuration menu
    Copy the full SHA
    47e8395 View commit details
    Browse the repository at this point in the history
  12. doc: add branch-diff example to releases.md

    Useful to have for reference, especially for onboarding
    
    PR-URL: nodejs#4636
    Reviewed-By: Colin Ihrig <[email protected]>
    Reviewed-By: James M Snell<[email protected]>
    Myles Borins committed Jan 13, 2016
    Configuration menu
    Copy the full SHA
    500477c View commit details
    Browse the repository at this point in the history
  13. docs: fix npm command in releases.md

    PR-URL: nodejs#4656
    Reviewed-By: Colin Ihrig <[email protected]>
    Reviewed-By: Rod Vagg <[email protected]>
    Reviewed-By: James M Snell <[email protected]>
    Myles Borins committed Jan 13, 2016
    Configuration menu
    Copy the full SHA
    ed427a6 View commit details
    Browse the repository at this point in the history
  14. docs: update gpg key for Myles Borins

    Original key did not have personal information associated with it.
    
    The new key rectifies this problem
    
    PR-URL: nodejs#4657
    Reviewed-By: Colin Ihrig <[email protected]>
    Reviewed-By: Rod Vagg <[email protected]>
    Reviewed-By: James M Snell <[email protected]>
    Myles Borins committed Jan 13, 2016
    Configuration menu
    Copy the full SHA
    2192fc8 View commit details
    Browse the repository at this point in the history

Commits on Jan 14, 2016

  1. test: remove unused modules

    Many tests use require() to import modules that subsequently never gets
    used. This removes those imports and, in a few cases, removes other
    unused variables from tests.
    Trott committed Jan 14, 2016
    Configuration menu
    Copy the full SHA
    0cac164 View commit details
    Browse the repository at this point in the history