Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 2, 2026

Summary

Removes dangling references to REORDER_FEATURE_SUMMARY.md from the Xcode project file. The file was referenced in commit 8057953 but never actually created, leaving orphaned entries in project.pbxproj.

Removed four entries:

  • PBXBuildFile resource reference
  • PBXFileReference definition
  • Home group children array entry
  • iOS App Resources build phase entry

Screenshots

N/A - Project configuration change only

Link to pull request in Documentation repository

N/A

Any other notes

None

Original prompt

Reference: 8057953

Remove REORDER_FEATURE_SUMMARY.md from project.pbxproj


💡 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 AI changed the title [WIP] Remove REORDER_FEATURE_SUMMARY.md from project.pbxproj Remove REORDER_FEATURE_SUMMARY.md from project.pbxproj Jan 2, 2026
Copilot AI requested a review from bgoncal January 2, 2026 17:36
@bgoncal bgoncal marked this pull request as ready for review January 2, 2026 17:47
Copilot AI review requested due to automatic review settings January 2, 2026 17:47
@bgoncal bgoncal merged commit 3c4b13d into copilot/add-homeview-ui Jan 2, 2026
2 checks passed
@bgoncal bgoncal deleted the copilot/remove-reorder-feature-summary branch January 2, 2026 17:47
@github-actions
Copy link
Contributor

github-actions bot commented Jan 2, 2026

⚠️ SwiftLint disabled in this PR

The following added lines contain // swiftlint:disable. Please verify this is necessary.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes four dangling references to REORDER_FEATURE_SUMMARY.md from the Xcode project file. The file was referenced in a previous commit but was never actually created, leaving orphaned entries that serve no purpose.

  • Removes all project file references to the non-existent REORDER_FEATURE_SUMMARY.md documentation file
  • Cleans up PBXBuildFile, PBXFileReference, group membership, and build phase entries
  • No functional impact - purely a project configuration cleanup

@github-actions
Copy link
Contributor

github-actions bot commented Jan 2, 2026

⚠️ Unused L10n strings detected

Found 1 unused localization strings in the codebase.

Click to see details
Parsing Strings.swift...
Found 1297 L10n strings

Reading all Swift source code...
Read 4900234 characters of Swift code

Checking for unused strings...
Checked 100/1297 strings...
Checked 200/1297 strings...
Checked 300/1297 strings...
Checked 400/1297 strings...
Checked 500/1297 strings...
Checked 600/1297 strings...
Checked 700/1297 strings...
Checked 800/1297 strings...
Checked 900/1297 strings...
Checked 1000/1297 strings...
Checked 1100/1297 strings...
Checked 1200/1297 strings...

================================================================================
UNUSED STRINGS REPORT
================================================================================

Found 1 unused strings:


WATCH:
  - L10n.Watch.Labels.noAction
    Key: watch.labels.no_action
    Line: 3984

================================================================================
Total unused: 1
================================================================================

To clean up these strings, manually remove them from the Localizable.strings files
and regenerate Strings.swift using SwiftGen.

@codecov
Copy link

codecov bot commented Jan 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (copilot/add-homeview-ui@8057953). Learn more about missing BASE report.

Additional details and impacted files
@@                    Coverage Diff                     @@
##             copilot/add-homeview-ui    #4150   +/-   ##
==========================================================
  Coverage                           ?   45.10%           
==========================================================
  Files                              ?      243           
  Lines                              ?    13890           
  Branches                           ?        0           
==========================================================
  Hits                               ?     6265           
  Misses                             ?     7625           
  Partials                           ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

2 participants