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

Create github pages with doxygen (and css) #1603

Merged
merged 2 commits into from
Jun 28, 2022

Conversation

scarf005
Copy link
Member

@scarf005 scarf005 commented Jun 7, 2022

Summary

SUMMARY: Infrastructure "Create github pages with doxygen (and css)"

Purpose of change

Found out there was an working doxyfile, so having online documentation readily available would be helpful. Of course the state of documentation is questionable but better late than never.

generated link would be https://cataclysmbnteam.github.io/Cataclysm-BN/ .

Describe the solution

Action works in following steps:

  1. checkout CBN repository (or in other words, docker container running this workflow clones it)
  2. check if there are cached css files.
  3. if not, clone doxygen-awesome-css
  4. run doxygen. generated yields are ~ 400mb.
  5. deploy to github pages, in branch gh-pages.

Describe alternatives you've considered

result output is humongously large (~300mb), which might hit hard limit on file size (1GB) and monthly bandwidth(100GB).

using some of technics like this https://stackoverflow.com/a/7455621/13503626 might help reducing release size.

Testing

checked the script to run on forked repo.

Additional context

below are some of the screenshots showing how generated documentation looks like:

_01
_02

@scarf005 scarf005 marked this pull request as draft June 7, 2022 11:10
@scarf005 scarf005 changed the title ci: doxygen support (with css) Create github pages for doxygen Jun 7, 2022
@scarf005 scarf005 changed the title Create github pages for doxygen Create github pages with doxygen (and css) Jun 7, 2022
there are working doxyfile, so why not?
@scarf005 scarf005 marked this pull request as ready for review June 7, 2022 12:10
@Coolthulhu Coolthulhu self-assigned this Jun 8, 2022
@Coolthulhu
Copy link
Member

result output is humongously large (~300mb), which might hit hard limit on file size (1GB) and monthly bandwidth(100GB).

How much of it has to be downloaded to view something?
Does viewing a random page require 10+mb of downloads?

@scarf005
Copy link
Member Author

deployed page on fork

network inspection says it takes around 20kb ~ 120kb per page. so sizes per each page are probably normal.

giant total size is due to it having 14000 files, mainly html, js, and png (mostly dependency graphs)

❯ ls | grep .html | wc -l
4720
❯ ls | grep .js | wc -l
2230
❯ ls | grep .png | wc -l
2926
❯ ls | wc -l
14613

page inspection result #1
page inspection result #2

@Coolthulhu Coolthulhu merged commit b80a8ba into cataclysmbnteam:upload Jun 28, 2022
@scarf005
Copy link
Member Author

oh hecc still forgor to add back the actual script, will post another pr

joveeater pushed a commit to joveeater/Cataclysm-BN that referenced this pull request Jul 10, 2022
* ci: doxygen support (with css)

there are working doxyfile, so why not?

* ci: fix permissions error
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