-
Notifications
You must be signed in to change notification settings - Fork 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
Publish reports from travis #66
Conversation
Please give me a bit of time to set up the io pages for private octopus. There is a bit of housekeeping needed first, and will probbaly need to do that tomorrow. |
I have created the repo at https://private-octopus.github.io/ithitools/. You have collaborator access to the ithitools project. What else do we need? |
I would prefer a different repository than private-octopus/ithitools , because we need to authorize our (own) travis job to push to the repository where the reports will be published. How about private-octopus/ithi-reports ? There is no rush either. We could sit down together at IETF and setup the authorization bits... |
It turns out that I won't be at the IETF next week. My wife's brother
passed away, and I will be traveling to France instead of Montreal.
Let's do a conference call after the 24.
…-- Christian Huitema
On 6/29/2018 8:12 AM, wtoorop wrote:
I would prefer a different repository than private-octopus/ithitools ,
because we need to authorize our (own) travis job to push to the
repository where the reports will be published.
How about private-octopus/ithi-reports ?
There is no rush either. We could sit down together at IETF and setup
the authorization bits...
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<#66 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AHHkwChKYh7IgdAwkPtUWvbjRoh2HdJXks5uBkPKgaJpZM4U8Lyc>.
|
Especially try not to use tab characters
Reports are now pushed to https://private-octopus.github.io/ithipages/#code-analysis-reports |
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 good. We may want to add some doxygen flags later, e.g. values and logo, but that's a start.
Please review (and merge if you want to accept) PR #65 before merging in this one, because this one contains all the commits also done in PR #65.
In addition to the changes in PR #64, this PR publishes reports from:
on a github pages site. Currently this is at: https://wtoorop.github.io/ithipages
We could change that location into for example: https://private-octopus.github.io/ithipages or https://private-octopus.github.io/ithi-reports , but for that you will have to add a repository and enable GitHub pages (it is on the Settings -> Options page, somewhere near the bottom)
If you give me access to that repository I can setup the key to publish from travis. Or you can do that yourself. I followed the procedure described in: https://blog.ionelmc.ro/2015/07/11/publishing-to-github-pages-from-travis-ci/