Skip to content

Remove repo-local dual test skill#853

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

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

Conversation

@Kyle-Ye
Copy link
Copy Markdown
Collaborator

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

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
Copy link
Copy Markdown

augmentcode bot commented Apr 18, 2026

🤖 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 👎

Copy link
Copy Markdown

@augmentcode augmentcode bot left a comment

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
Copy link
Copy Markdown

codecov bot commented Apr 18, 2026

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