-
Notifications
You must be signed in to change notification settings - Fork 0
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
Install flow tool for static analysis #52
Conversation
Pull Request Test Coverage Report for Build 11654082662Warning: This coverage report may be inaccurate.This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Details
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have incorporated the changes made, installed the tool, and ran it. The output on my machine matches the one here. LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The tool installation looks good and functions as it should.
Static analysis tool
Flow
is installed in the codebase through the commandnpm install --save-dev flow-bin
. To configure theNode.js
codebase to use the tool, aFlow
script is added inpackage.json
. The tool was then initialized through the commandnpm run flow init
. This created a.flowconfig
file that defines the tool's configuration to work correctly within the codebase. Additionally, the commandnpm run flow
was deployed to launch the Flow server and confirm that no errors were found in the codebase.The following screenshot of the terminal shows the output of running
npm run flow init
and executingnpm run flow
.