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

ci: replace generated headers by templates #723

Merged
merged 1 commit into from
Oct 12, 2021
Merged

Conversation

sergiud
Copy link
Collaborator

@sergiud sergiud commented Oct 11, 2021

Currently, coverage analysis runs particularly on generated headers. However, since these are not readily available for Coveralls their source cannot be displayed prohibiting their online inspection.

This PR patches the coverage files by replacing the paths of generated headers by their template sources.

@sergiud sergiud added this to the 0.6 milestone Oct 11, 2021
@google-cla google-cla bot added the cla: yes label Oct 11, 2021
@sergiud
Copy link
Collaborator Author

sergiud commented Oct 11, 2021

@drigz I got in touch with the Coveralls support team because the PR comments still do not appear. They were very helpful at tracing the problem to a disabled setting in the web interface. Specifically, the PR comments must be enabled at https://coveralls.io/github/google/glog/settings under "Pull Requests Alerts":
Screen Shot 2021-10-11 at 1 29 17 PM
Could you please check the setting and enable it as necessary?

.github/workflows/linux-builds.yml Outdated Show resolved Hide resolved
@sergiud sergiud force-pushed the coverage-use-templates branch from 8250c00 to 06e6651 Compare October 12, 2021 07:38
@coveralls
Copy link
Collaborator

coveralls commented Oct 12, 2021

Coverage Status

Coverage increased (+0.2%) to 73.116% when pulling 06e6651 on coverage-use-templates into 56d3fac on master.

@sergiud
Copy link
Collaborator Author

sergiud commented Oct 12, 2021

@drigz Thanks!

@sergiud sergiud merged commit ce35336 into master Oct 12, 2021
@sergiud sergiud deleted the coverage-use-templates branch October 12, 2021 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants