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

Revise for consistency and readability #188

Closed
wants to merge 9 commits into from
Closed

Revise for consistency and readability #188

wants to merge 9 commits into from

Commits on Apr 15, 2019

  1. move declaration of changelog up with the others

    @elidoran
    elidoran committed Apr 15, 2019
    Configuration menu
    Copy the full SHA
    5061598 View commit details
    Browse the repository at this point in the history

  2. change from const to let so we can color them if needed

    @elidoran
    elidoran committed Apr 15, 2019
    Configuration menu
    Copy the full SHA
    26f60fd View commit details
    Browse the repository at this point in the history

  3. create variables for remaining parts of output text

    @elidoran
    elidoran committed Apr 15, 2019
    Configuration menu
    Copy the full SHA
    4442107 View commit details
    Browse the repository at this point in the history

  4. colorize all variables inside single block

    @elidoran
    elidoran committed Apr 15, 2019
    Configuration menu
    Copy the full SHA
    a255363 View commit details
    Browse the repository at this point in the history

  5. eliminate unnecessary useColor ternary statements

    @elidoran
    elidoran committed Apr 15, 2019
    Configuration menu
    Copy the full SHA
    3f8e663 View commit details
    Browse the repository at this point in the history

  6. remove remaining ternary statement by moving it to a variable; arrow

    @elidoran
    elidoran committed Apr 15, 2019
    Configuration menu
    Copy the full SHA
    c01af1a View commit details
    Browse the repository at this point in the history

  7. useColor is used once now so move its value to the if statement

    @elidoran
    elidoran committed Apr 15, 2019
    Configuration menu
    Copy the full SHA
    9dd157f View commit details
    Browse the repository at this point in the history

  8. reformat notify message construction now that all the ternary express… 

    …ions are gone
    @elidoran
    elidoran committed Apr 15, 2019
    Configuration menu
    Copy the full SHA
    f1b1acd View commit details
    Browse the repository at this point in the history

  9. move useUnicode declaration up a scope and reuse it below

    @elidoran
    elidoran committed Apr 15, 2019
    Configuration menu
    Copy the full SHA
    d9008c5 View commit details
    Browse the repository at this point in the history