Skip to content

Remove repo-local dual test skill - #853

Merged
Kyle-Ye merged 1 commit into
mainfrom
codex/remove-local-dual-test-skill
Apr 19, 2026
Merged

Remove repo-local dual test skill#853
Kyle-Ye merged 1 commit into
mainfrom
codex/remove-local-dual-test-skill

Conversation

@Kyle-Ye

@Kyle-Ye Kyle-Ye commented Apr 18, 2026

Copy link
Copy Markdown
Member

Summary

  • Remove the repo-local .claude/skills/add-dual-test-case skill now that its guidance lives in the shared OpenSwiftUI workflow skill.

Testing

  • Not run; documentation/skill cleanup only.

@augmentcode

augmentcode Bot commented Apr 18, 2026

Copy link
Copy Markdown
🤖 Augment PR Summary

Summary: This PR removes the repository-local Claude skill for adding DualTests now that the guidance is maintained in the shared OpenSwiftUI workflow.

Changes:

  • Deleted .claude/skills/add-dual-test-case/SKILL.md containing the DualTests rules and templates
  • Consolidated maintenance of this guidance into the shared workflow skill to avoid duplication/drift

Technical Notes: No production or test code changes; this is a documentation/automation cleanup.

🤖 Was this summary useful? React with 👍 or 👎

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

@codecov

codecov Bot commented Apr 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 14.41%. Comparing base (8b4076e) to head (114d7f6).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #853   +/-   ##
=======================================
  Coverage   14.41%   14.41%           
=======================================
  Files         634      634           
  Lines       39006    39006           
=======================================
+ Hits         5621     5623    +2     
+ Misses      33385    33383    -2     

☔ 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.

@Kyle-Ye
Kyle-Ye merged commit c2abbf4 into main Apr 19, 2026
4 of 10 checks passed
@Kyle-Ye
Kyle-Ye deleted the codex/remove-local-dual-test-skill branch April 19, 2026 06:53
@Kyle-Ye Kyle-Ye added area: tests Test suites, UI tests, fixtures, snapshots, and test support code. type: tests Test coverage, test infrastructure, snapshots, or test-only changes. labels May 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: tests Test suites, UI tests, fixtures, snapshots, and test support code. type: tests Test coverage, test infrastructure, snapshots, or test-only changes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant