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

Port some commits from from joyent/node@master #1770

Merged
merged 2 commits into from
Jun 1, 2015

Commits on Jun 1, 2015

  1. docs: delete unused/duplicate css files

     - `sh.css` already exists in `api_assets`
     - `sh_vim-dark.css` is unused, but used in the repo `node-website`
            now
    
    Reviewed-by: Trevor Norris <trev.norris@gmail.com>
    Signed-off-by: Julien Gilli <julien.gilli@joyent.com>
    
    PORT-FROM: joyent/node @ 0c50195
    PR-URL: nodejs#1770
    Reviewed-By: Roman Reiss <me@silverwind.io>
    Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
    robertkowalski authored and Fishrock123 committed Jun 1, 2015
    Configuration menu
    Copy the full SHA
    1cb72c1 View commit details
    Browse the repository at this point in the history
  2. docs: add return value for sync fs functions

    Clarify that synchronous functions in fs with no return value return
    undefined.
    
    Specify that fs.openSync() returns an integer and fs.existsSync()
    returns true or false.
    
    Fixes: nodejs/node-v0.x-archive#9313
    
    PR: nodejs/node-v0.x-archive#9359
    Reviewed-By: Julien Gilli <julien.gilli@joyent.com>
    
    PORT-FROM: joyent/node @ 51fe319
    PR-URL: nodejs#1770
    Reviewed-By: Roman Reiss <me@silverwind.io>
    Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
    
    Conflicts:
    	doc/api/fs.markdown
    tyleranton authored and Fishrock123 committed Jun 1, 2015
    Configuration menu
    Copy the full SHA
    a79dece View commit details
    Browse the repository at this point in the history