-
Notifications
You must be signed in to change notification settings - Fork 257
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
Tech timeout 20 hours or greater #4058
Conversation
✅ Deploy Preview for wcag2 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Discussed on TF call 9/13. This would be new technique from a stale branch. TBD. |
@goodwitch are you still of the opinion that this is an important gap to fill? That is, do we need a technique for testing that a timeout does not occur within 20 hours? And if so, is what @patrickhlauke has drafted sufficient? Thank you. |
to clarify, this is literally just taking the branch that @goodwitch opened at the time and turning it into a PR. it will need massaging to be mergeable |
On TF call 10/4, @alastc offered to ask @goodwitch offline about the need for this PR. |
I would suggest closing this (and deleting the branch), as we likely don't need a technique that literally says "to test there's a 20 hour timeout, set yourself a timer for 19 hours 59 minutes and check if anything happens in that last minute..." |
Discussed on today's TF call. Attempts were made to contact creator. Closing as unnecessary. |
This branch never had a pull request made. Suggest we either take/consider this, or delete both this PR and branch.