-
Notifications
You must be signed in to change notification settings - Fork 51
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
V3: Use xplatform for logging #4071
Conversation
Signed-off-by: Martin Zeithaml <[email protected]>
Signed-off-by: Martin Zeithaml <[email protected]>
build 6408 FAILED. |
build 6409 FAILED. |
build 6410 FAILED. |
1 similar comment
build 6410 FAILED. |
Signed-off-by: JoeNemo <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
v2 has exactly this code so somehow this was missed for v3. thanks martin!
build 6464 FAILED. |
Signed-off-by: Martin Zeithaml <[email protected]>
Signed-off-by: Martin Zeithaml <[email protected]>
build 6468 SUCCEEDED. |
Test workflow 5703 is started. |
Signed-off-by: Martin Zeithaml <[email protected]>
build 6471 SUCCEEDED. |
Test workflow 5706 is started. |
Signed-off-by: Martin Zeithaml <[email protected]>
build 6511 SUCCEEDED. |
Test workflow 5733 is started. |
Please check if your PR fulfills the following requirements. This is simply a reminder of what we are going to look for before merging your PR. If you don't know all of this information when you create this PR, don't worry. You can edit this template as you're working on it.
PR type
What type of changes does your PR introduce to Zowe? Put an
x
in the box that applies to this PR. If you're unsure about any of them, don't hesitate to ask.Relevant issues
Fixes #4062
Changes proposed in this PR
Does this PR introduce a breaking change?