Skip to content

566 logs #743

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

Merged
merged 34 commits into from
Oct 25, 2017
Merged

566 logs #743

merged 34 commits into from
Oct 25, 2017

Conversation

bmuenzenmeyer
Copy link
Member

@bmuenzenmeyer bmuenzenmeyer commented Oct 25, 2017

Addresses #566 - Refactor All Logs

Summary of changes:

  • Rename utilities.js to log.js
  • Added info level to log
  • Users can now use a new key within patternlab-config.json, logLevel, to opt into information.
    • error will output a message as a thrown error
    • warning will output all warnings plus above
    • info will output all info messages, plus above (intended default)
    • debug will output all debug messages, plus above
    • quiet will output ZERO logs
  • Pattern Lab will warn you if it detects the old debug flag, and attempt to set some appropriate defaults (error, warning, info)
  • logLevel, if present, will supercede debug

…, 'error', as well as 'quiet'

backwards compatible with boolean debug
@bmuenzenmeyer bmuenzenmeyer added the hacktoberfest 🌾 https://hacktoberfest.digitalocean.com label Oct 25, 2017
@coveralls
Copy link

coveralls commented Oct 25, 2017

Coverage Status

Changes Unknown when pulling 7316d37 on 566-logs into ** on dev**.

@coveralls
Copy link

coveralls commented Oct 25, 2017

Coverage Status

Changes Unknown when pulling 1a1ef62 on 566-logs into ** on dev**.

@bmuenzenmeyer bmuenzenmeyer merged commit fca4d03 into dev Oct 25, 2017
@bmuenzenmeyer bmuenzenmeyer deleted the 566-logs branch October 25, 2017 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest 🌾 https://hacktoberfest.digitalocean.com
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants