Skip to content
This repository was archived by the owner on Apr 23, 2025. It is now read-only.

Conversation

sbouchet
Copy link
Collaborator

@sbouchet sbouchet commented Jul 9, 2021

Signed-off-by: Stephane Bouchet [email protected]

What is the purpose of this change? What does it change?

fixes NPE when sending telemetry error.

Was the change discussed in an issue?

fixes #???

How to test changes?

Signed-off-by: Stephane Bouchet <[email protected]>
@sbouchet sbouchet requested a review from jeffmaury July 9, 2021 08:06
@sbouchet
Copy link
Collaborator Author

sbouchet commented Jul 9, 2021

tht's actually a workaround for redhat-developer/intellij-redhat-telemetry#23

Copy link
Member

@jeffmaury jeffmaury left a comment

Choose a reason for hiding this comment

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

LGTM

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 9, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@sbouchet sbouchet merged commit 6681926 into redhat-developer:main Jul 9, 2021
@sbouchet sbouchet deleted the fix_telemetry branch July 9, 2021 12:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants