ably-cocoa icon indicating copy to clipboard operation
ably-cocoa copied to clipboard

[Testing copilot agent] Remove unused deprecated logExceptionReportingUrl property

Open Copilot opened this issue 8 months ago • 0 comments

This PR removes the deprecated and unused logExceptionReportingUrl property from ARTClientOptions as identified in the issue.

Changes Made

  • Removed property declaration from ARTClientOptions.h including its deprecation comment
  • Removed property initialization from the initDefaults method in ARTClientOptions.m
  • Removed property copying from the copyWithZone: method in ARTClientOptions.m

Verification

  • ✅ Confirmed the property was only referenced in these two files
  • ✅ Verified no tests or other code depends on this property
  • ✅ Ensured no references remain anywhere in the codebase
  • ✅ Basic syntax validation confirms the changes don't break file structure

Impact

This is a safe removal as:

  • The property was already marked as DEPRECATED with a note that it "will be removed in a future version"
  • No code in the repository actually uses this property beyond initialization and copying
  • The removal eliminates 7 lines of dead code (2 from .m file, 5 from .h file)

Fixes #2025.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Copilot avatar Jun 11 '25 10:06 Copilot