Skip to content

rts test: Name the tests of properties by whether they are editable - #4221

Open
marksvc wants to merge 2 commits into
task/sec1-5from
task/sec1-6
Open

marksvc wants to merge 2 commits into
task/sec1-5from
task/sec1-6

Conversation

@marksvc

@marksvc marksvc commented Oct 22, 2026 •

Copy link
Copy Markdown
Collaborator

Open in Devin Review


Stack created with GitHub Stacks CLI • Give Feedback 💬


This change is Reviewable

@marksvc
marksvc added this pull request to stack #4226 October 22, 2026 21:54
@codecov

codecov Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.20%. Comparing base (40ac8c0) to head (7ac6eea).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff              @@
##           task/sec1-5    #4221   +/-   ##
============================================
  Coverage        81.20%   81.20%           
============================================
  Files              670      670           
  Lines            42920    42920           
  Branches          7116     7116           
============================================
  Hits             34853    34853           
  Misses            6919     6919           
  Partials          1148     1148           

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

@marksvc
marksvc deployed to screenshot_diff October 22, 2026 22:10 — with GitHub Actions Active
@marksvc marksvc added testing not required ai-review-author-sign-off PR author has looked at AI review of change and determined no findings warrant further changes labels Oct 23, 2026
@marksvc
marksvc marked this pull request as ready for review October 23, 2026 15:58
@marksvc
marksvc removed this pull request from stack #4226 October 23, 2026 16:11
@marksvc
marksvc added this pull request to stack #4228 October 23, 2026 16:11
@marksvc
marksvc deployed to screenshot_diff October 23, 2026 16:16 — with GitHub Actions Active
@github-actions

Copy link
Copy Markdown
Contributor

📸 Screenshot diff deployed! (118 changes)

View the visual diff at: https://pr-4221--sf-screenshot-diffs.netlify.app

This branch was successfully deployed

1 active deployment
screenshot_diff — 7ac6eea4 Deployed Oct 23, 2026 by marksvc via Compare Screenshots #1187
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-review-author-sign-off PR author has looked at AI review of change and determined no findings warrant further changes testing not required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant