Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Retain github manual edit warning #1293

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

wx-03
Copy link

@wx-03 wx-03 commented Jan 16, 2025

Summary:

Fixes #1021

Changes Made:

  • Added github manual edit warning to tester response so that the warning remains on the github comment when tester edits/submits a response
  • Updated test code to reflect new changes

Proposed Commit Message:

Retain github manual edit warning when editing/submitting in tester response phase

Github manual edit warning message disappears after editing/submitting in tester response phase.

The warning message should remain in the issue comment to warn testers against editing directly their response on GitHub directly.

@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 58.46%. Comparing base (5e7ed48) to head (bdf33b3).
Report is 13 commits behind head on master.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1293      +/-   ##
==========================================
+ Coverage   54.82%   58.46%   +3.64%     
==========================================
  Files         105      104       -1     
  Lines        2858     2574     -284     
  Branches      503      290     -213     
==========================================
- Hits         1567     1505      -62     
+ Misses       1030     1017      -13     
+ Partials      261       52     -209     

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

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.

github manual edit warning message disappears after editing/submitting in tester response phase
2 participants