Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Suppress telemetry #1021

Merged
merged 8 commits into from
Feb 4, 2025
Merged

Suppress telemetry #1021

merged 8 commits into from
Feb 4, 2025

Conversation

Arjun-Zingg
Copy link
Contributor

No description provided.

@@ -125,7 +133,7 @@
String response = executePostRequest(url.toString(), param);
LOG.warn("Analytics event " + response);
} catch (IOException | URISyntaxException e) {
e.printStackTrace();
if(LOG.isDebugEnabled()) e.printStackTrace();

Check warning

Code scanning / PMD

This statement should have braces

This statement should have braces
@sonalgoyal sonalgoyal merged commit 50332cb into zinggAI:main Feb 4, 2025
3 of 4 checks passed
@Arjun-Zingg Arjun-Zingg deleted the suppress-telemetry branch February 4, 2025 03:46
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants