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

Cli #769

Closed
wants to merge 5 commits into from
Closed

Cli #769

wants to merge 5 commits into from

Conversation

nspringman
Copy link
Collaborator

Related issues and PRs

  • Begin working more towards cli
    So far based on issue Introduce CLI and NPM package #663, we have begun
    -Added cli/ directory (and changed package.json to reflect that)
    -Added the neodoc parser
    -Wrapped run() in an exportable function
    -Read severity and age distribution json files instead of "import get*" statements
    -^and doing the above with fs-extra
    -Switching console.log to console.info/warn

Description

Impacted Areas in the application

Testing

@vercel
Copy link

vercel bot commented Jul 27, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/neherlab/covid19-scenarios/qd2yr5gj3
✅ Preview: https://covid19-scenarios-git-fork-437su20-c-r-n-cli.neherlab.vercel.app

*
* @param inputFilename The path to the file.
*/
function getSeverity(inputFilename: string | undefined) {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Similar blocks of code found in 2 locations. Consider refactoring.

* @param name The age distribution name to use if no file is
* specified.
*/
function getAge(inputFilename: string | undefined, name: string) {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Similar blocks of code found in 2 locations. Consider refactoring.

@codeclimate
Copy link

codeclimate bot commented Jul 27, 2020

Code Climate has analyzed commit 6f7d256 and detected 2 issues on this pull request.

Here's the issue category breakdown:

Category Count
Duplication 2

View more on Code Climate.

@nspringman nspringman closed this Jul 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant