Skip to content

Add comprehensive documentation to achieve 100% coverage.#7

Merged
samuel-williams-shopify merged 1 commit intomainfrom
feature/add-comprehensive-documentation
Aug 26, 2025
Merged

Add comprehensive documentation to achieve 100% coverage.#7
samuel-williams-shopify merged 1 commit intomainfrom
feature/add-comprehensive-documentation

Conversation

@samuel-williams-shopify
Copy link
Contributor

@samuel-williams-shopify samuel-williams-shopify commented Aug 26, 2025

  • Added detailed documentation for all 8 interface methods with proper @parameter, @returns, and @yields pragmas
  • Documented Interface module with clear description
  • Added @namespace pragmas to organizational modules in version.rb
  • Achieved 12/12 definitions documented (100% coverage)
  • All documentation follows project standards from decode gem guidelines

Methods documented:

  • trace: Creates trace spans with name and attributes
  • trace_context=: Sets current trace context
  • active?: Checks for active trace span
  • trace_context: Gets current trace context
  • current_context: Gets OpenTelemetry context
  • with_context: Executes code within context
  • inject: Injects trace context into headers
  • extract: Extracts trace context from headers

Types of Changes

  • Maintenance.

Contribution

- Added detailed documentation for all 8 interface methods with proper @parameter, @returns, and @yields pragmas
- Documented Interface module with clear description
- Added @namespace pragmas to organizational modules in version.rb
- Achieved 12/12 definitions documented (100% coverage)
- All documentation follows project standards from decode gem guidelines

Methods documented:
- trace: Creates trace spans with name and attributes
- trace_context=: Sets current trace context
- active?: Checks for active trace span
- trace_context: Gets current trace context
- current_context: Gets OpenTelemetry context
- with_context: Executes code within context
- inject: Injects trace context into headers
- extract: Extracts trace context from headers
@samuel-williams-shopify samuel-williams-shopify merged commit c82e520 into main Aug 26, 2025
38 of 40 checks passed
@samuel-williams-shopify samuel-williams-shopify deleted the feature/add-comprehensive-documentation branch August 26, 2025 08:48
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.

1 participant