Skip to content

log error info to stderr - #750

Merged
mukundansundar merged 7 commits into
dapr:masterfrom
daixiang0:error
Aug 25, 2021
Merged

log error info to stderr#750
mukundansundar merged 7 commits into
dapr:masterfrom
daixiang0:error

Conversation

@daixiang0

Copy link
Copy Markdown
Member

Signed-off-by: Long Dai long0dai@foxmail.com

Description

Log error info to stderr.

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #748

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

@codecov

codecov Bot commented Jul 2, 2021

Copy link
Copy Markdown

Codecov Report

Merging #750 (b90ab4b) into master (bd39e6f) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #750   +/-   ##
=======================================
  Coverage   20.73%   20.73%           
=======================================
  Files          29       29           
  Lines        1476     1476           
=======================================
  Hits          306      306           
  Misses       1133     1133           
  Partials       37       37           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bd39e6f...b90ab4b. Read the comment docs.

wcs1only
wcs1only previously approved these changes Jul 3, 2021
@daixiang0

Copy link
Copy Markdown
Member Author

ping @artursouza @yaron2

@mukundansundar

Copy link
Copy Markdown
Collaborator

@daixiang0 can you comment on the issue so that it can be assigned to you?

mukundansundar
mukundansundar previously approved these changes Jul 14, 2021
@daixiang0

Copy link
Copy Markdown
Member Author

@mukundansundar done.

artursouza
artursouza previously approved these changes Jul 28, 2021
daixiang0 and others added 2 commits July 29, 2021 10:46
Signed-off-by: Long Dai <long0dai@foxmail.com>
@mukundansundar
mukundansundar requested review from a team as code owners August 3, 2021 10:57
@mukundansundar
mukundansundar merged commit 909b989 into dapr:master Aug 25, 2021
@daixiang0
daixiang0 deleted the error branch August 25, 2021 06:50
anhldbk pushed a commit to anhldbk/dapr-cli that referenced this pull request Dec 22, 2021
Signed-off-by: Long Dai <long0dai@foxmail.com>

Co-authored-by: Mukundan Sundararajan <musundar@microsoft.com>
Co-authored-by: Artur Souza <artursouza.ms@outlook.com>
Signed-off-by: Andy Le <anhldbk@gmail.com>
anhldbk pushed a commit to anhldbk/dapr-cli that referenced this pull request Dec 22, 2021
Signed-off-by: Long Dai <long0dai@foxmail.com>

Co-authored-by: Mukundan Sundararajan <musundar@microsoft.com>
Co-authored-by: Artur Souza <artursouza.ms@outlook.com>
Signed-off-by: Andy Le <anhldbk@gmail.com>
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.

An error messages must write to standard error.

4 participants