-
Notifications
You must be signed in to change notification settings - Fork 0
Program Testing Strategy #6
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
base: main
Are you sure you want to change the base?
Conversation
|
After our conversation today, this likely needs to be adjusted and scoped to something actionable and inline with the program architecture refactor |
| - Add coverage tooling | ||
| * Add `coverage.py` for BE coverage reporting | ||
| * Add GH workflow to upload BE coverage report to CodeCov | ||
| * Add GH workflow to upload FE coverage report to CodeCov |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I could see an argument for moving this to Phase 2 if we wanted to get to the new architecture and TX implementation ASAP.
I have it in Phase 1 because we want our TX implementation to be exemplary and I think it'll help us ensure our tests are comprehensive.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, excited for this! Left a few thoughts.
73f985b to
4d2062f
Compare
RFC Pull Request
RFC Number: 0008
RFC Title: Program Testing Strategy
Status: Discussion
Summary
Checklist
By submitting this RFC, I agree to engage constructively with feedback and work toward consensus.