-
Notifications
You must be signed in to change notification settings - Fork 88
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
Setup codecov #284
Setup codecov #284
Conversation
@chepsi still working on this. Don't understand why it automatically assigns reviewers |
@Raynafs the reviewers are requested automatically because of Code Owners, you can however convert the PR to draft if it is not ready for review |
Thanks @kibettheophilus |
@Raynafs I have fixed the failing CI with my latest merge please rebase from develop |
@chepsi @kibettheophilus This is ready, just needs to be tested in codecov. |
Scope
Please make sure to read the Contribution Guidelines
and check that you understand and have followed it as best as possible Explain what your feature
does in a short paragraph. please check the below boxes
./codeAnalysis.sh
on linux/unix orcodeAnalysys.bat
on windows to make sure all lint/formatting checks have been done.#283
Declare any issues by typing
fixes #1
orcloses #1
for example so that the automation can kickin when this is merged
Other testing QA Notes
What have you tested specifically and what possible impacts/areas there are that may need retesting
by others.
Please add a screenshot (if necessary)