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 crate docs to GitHub Pages #86

Closed
wants to merge 5 commits into from

Conversation

bavshin-f5
Copy link
Member

@bavshin-f5 bavshin-f5 commented Jul 27, 2024

docs.rs builds are currently broken (#85), and getting these fixed seems a bit complicated. The next best thing is to host the crate documentation ourselves on GitHub Pages.

Preview: https://bavshin-f5.github.io/ngx-rust/ngx/index.html
Only nightly docs for now, will figure out how to handle the tagged releases later.

Required repository configuration:

  • Enable pages for the repo (do we need to ask org admin?)
  • Settings -> Pages -> Source = Github Actions
  • Settings -> Environments -> github-pages -> Deployment branches and tags = master (should be created and configured automatically on the first run).

@bavshin-f5 bavshin-f5 marked this pull request as draft July 29, 2024 15:58
@xeioex
Copy link
Contributor

xeioex commented Oct 24, 2024

Hi @dekobon, can you help us with enabling Github pages for ngx-rust docs?

@bavshin-f5
Copy link
Member Author

Hi @dekobon, can you help us with enabling Github pages for ngx-rust docs?

We already chatted about that; pages are disabled on the org level and syseng team is not interested in changing that.
I plan to remove the CI parts from this pr and take a different approach on fixing the documentation build.

Copy link
Contributor

@xeioex xeioex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Documentation for ngx::log looks to empty. Can we re-export the macros like ngx_log_debug in the src/log.rs so they appear in the docs for ngx::log too?

Otherwise looks good.

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