Skip to content

Conversation

Copy link

Copilot AI commented Jun 11, 2025

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 AI changed the title [WIP] Remove not used var Remove unused deprecated logExceptionReportingUrl property Jun 11, 2025
Copilot AI requested a review from jamienewcomb June 11, 2025 10:51
@jamienewcomb jamienewcomb changed the title Remove unused deprecated logExceptionReportingUrl property [Testing copilot agent] Remove unused deprecated logExceptionReportingUrl property Jun 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Remove not used var

2 participants