Skip to content

Conversation

Copilot
Copy link

@Copilot Copilot AI commented Aug 25, 2025

Updates the span-stacktrace module to comply with the OpenTelemetry Java Contrib style guide requirements by making minimal, targeted changes:

Changes Made

  1. Added final keyword to public non-internal classes:

    • StackTraceSpanProcessor - now declared as public final class
    • StackTraceAutoConfig - now declared as public final class
  2. Added static imports for commonly used constants:

    • Static imports for Level.FINE and Level.SEVERE in StackTraceAutoConfig
    • Removed redundant Level import and updated usage to use static imports

These changes align with the style guide requirements that:

  • Public non-internal classes should be declared final where possible
  • Static imports should be used for commonly used constants like logging levels

All existing functionality is preserved and tests continue to pass. The changes are minimal and surgical, focusing only on style guide compliance without affecting behavior.

Fixes #413.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@Copilot Copilot AI changed the title [WIP] Update span-stacktrace module to follow style guide Update span-stacktrace module to follow style guide Aug 25, 2025
@Copilot Copilot AI requested a review from trask August 25, 2025 02:02
Copilot finished work on behalf of trask August 25, 2025 02:02
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.

Update span-stacktrace module to follow style guide
2 participants