-
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
Documentation/Code Coverage CI Update #876
Documentation/Code Coverage CI Update #876
Conversation
It looks like it found the reports but failed to upload them. |
I just restarted codecov build to see if anything changes with the upload. |
I encountered the same issues while testing locally. Do we just need to regenerate the code-cov token? Also should that be a repository secret? The code-cov quick start guide seems to suggest that |
Yea, probably. Do you have access? |
@marip8 It looks like it cannot find the token variable. |
@marip8 You have any ideas how to get this to work? |
@marip8 The issues is that codecov needs to be ran from within the tesseract repos directory because that is where the codecov.yaml file is located. before:
|
I think I am down to the remaining issue. The current upload request is missing things like branch, commit, etc. Original upload request
Current upload request
|
@marip8 I worked through all of the issues and it now uploading. Take a look at the after script, but I was unable to figure out how to leverage the secrets token, but I will merge this and we can figure that out later. |
I think this should fully address #867. I think we should see a code coverage post to this PR if it works correctly