Skip to content

Use WARN as the default error-reporting logging level#2237

Merged
DominicKramer merged 5 commits intogoogleapis:masterfrom
DominicKramer:bug/2236-warn-by-default
Apr 20, 2017
Merged

Use WARN as the default error-reporting logging level#2237
DominicKramer merged 5 commits intogoogleapis:masterfrom
DominicKramer:bug/2236-warn-by-default

Conversation

@DominicKramer
Copy link
Contributor

Fixes #2236: If the logging level is not specified when configuring the error-reporting library, warning messages are not displayed to the user.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 18, 2017
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 12b1131 on DominicKramer:bug/2236-warn-by-default into 55d4d5f on GoogleCloudPlatform:master.

@stephenplusplus stephenplusplus requested review from ofrobots and removed request for stephenplusplus April 19, 2017 12:17
}
};
});
afterEach(function() {

This comment was marked as spam.

The `.jshintignore` and `.jshintrc` files in this directory were
conflicting with the `.jshintignore` and `.jshintrc` files in
the `google-cloud-node` repo's root directory.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling e477f36 on DominicKramer:bug/2236-warn-by-default into 55d4d5f on GoogleCloudPlatform:master.

Copy link
Contributor

@ofrobots ofrobots left a comment

Choose a reason for hiding this comment

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

LGTM w/ nit.

assert.strictEqual(text,
'WARN:@google/cloud-errors: test warning message');
});
it('Should be able to ERROR by default', function() {

This comment was marked as spam.

This comment was marked as spam.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 2d61b70 on DominicKramer:bug/2236-warn-by-default into 55d4d5f on GoogleCloudPlatform:master.

@DominicKramer DominicKramer merged commit e3341bc into googleapis:master Apr 20, 2017
@DominicKramer DominicKramer deleted the bug/2236-warn-by-default branch April 20, 2017 22:08
GautamSharda pushed a commit that referenced this pull request Jan 14, 2026
Fixes #2236:  If the logging level is not specified when configuring the `error-reporting` library, warning messages are not displayed to the user.
GautamSharda pushed a commit that referenced this pull request Jan 15, 2026
Fixes #2236:  If the logging level is not specified when configuring the `error-reporting` library, warning messages are not displayed to the user.
GautamSharda pushed a commit that referenced this pull request Feb 5, 2026
Fixes #2236:  If the logging level is not specified when configuring the `error-reporting` library, warning messages are not displayed to the user.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: error-reporting cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants