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

Publish reports from travis #66

Merged
merged 16 commits into from
Aug 24, 2018
Merged

Publish reports from travis #66

merged 16 commits into from
Aug 24, 2018

Conversation

wtoorop
Copy link
Collaborator

@wtoorop wtoorop commented Jun 28, 2018

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:

  • static analysis
  • code coverage
  • doxygen documentation (only with pushes to the master branch)

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/

@wtoorop wtoorop requested a review from huitema June 28, 2018 22:46
@huitema
Copy link
Collaborator

huitema commented Jun 29, 2018

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.

@huitema
Copy link
Collaborator

huitema commented Jun 29, 2018

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?

@wtoorop
Copy link
Collaborator Author

wtoorop commented Jun 29, 2018

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...

@huitema
Copy link
Collaborator

huitema commented Jul 13, 2018 via email

@wtoorop
Copy link
Collaborator Author

wtoorop commented Aug 17, 2018

Reports are now pushed to https://private-octopus.github.io/ithipages/#code-analysis-reports
This PR is ready to be merged now

Copy link
Collaborator

@huitema huitema left a 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.

@huitema huitema merged commit 4e3693a into master Aug 24, 2018
@huitema huitema deleted the travis/reports branch August 24, 2018 02:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants