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

build: add cpp linting to windows build #11856

Closed
wants to merge 4 commits into from
Closed

build: add cpp linting to windows build #11856

wants to merge 4 commits into from

Commits on Mar 15, 2017

  1. build: add cpp linting to windows build 

    This PR adds cpp linting to windows build script. After this change,
running command `vcbuild lint` will run both cpp linting and javascript
linting on a windows machine.

Fixes: nodejs#11816
    liusi committed Mar 15, 2017
    Configuration menu
    Copy the full SHA
    3f6c5a9 View commit details
    Browse the repository at this point in the history

  2. remove accidentally added new line

    liusi committed Mar 15, 2017
    Configuration menu
    Copy the full SHA
    15ae7c0 View commit details
    Browse the repository at this point in the history

Commits on Mar 16, 2017

  1. make file path comparison work for windows machine as well.

    liusi committed Mar 16, 2017
    Configuration menu
    Copy the full SHA
    3822b82 View commit details
    Browse the repository at this point in the history

  2. indent by 4 spaces instead.

    liusi committed Mar 16, 2017
    Configuration menu
    Copy the full SHA
    1743f76 View commit details
    Browse the repository at this point in the history