Skip to content

common: Add log levels field to logger - #1888

Merged
stephenplusplus merged 2 commits into
googleapis:masterfrom
kjin:logLevels
Dec 19, 2016
Merged

common: Add log levels field to logger#1888
stephenplusplus merged 2 commits into
googleapis:masterfrom
kjin:logLevels

Conversation

@kjin

@kjin kjin commented Dec 17, 2016

Copy link
Copy Markdown
Contributor

There should be a way to get the list of possible log levels. Here it's just a member of logger.js exports.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Dec 17, 2016

@stephenplusplus stephenplusplus left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me with a couple small comments. Can you add a test as well?

/**
* A list of log levels.
* @type {Array}
*/

This comment was marked as spam.

var logDriver = require('log-driver');

/**
* A list of log levels.

This comment was marked as spam.

@stephenplusplus
stephenplusplus merged commit 388a2c4 into googleapis:master Dec 19, 2016
@stephenplusplus

Copy link
Copy Markdown
Contributor

LGTM. Thanks!

GautamSharda pushed a commit that referenced this pull request Mar 11, 2026
feywind pushed a commit that referenced this pull request Mar 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement. core

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants