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

Adding GH pages functionality #365

Merged
merged 19 commits into from Aug 18, 2023
Merged

Adding GH pages functionality #365

merged 19 commits into from Aug 18, 2023

Conversation

kbonney
Copy link
Collaborator

@kbonney kbonney commented Jul 20, 2023

Summary

Added an action that builds the documentation and deploys it to the github pages url for the repo. This also required updating the requirements and configuration for the documentation, as well as adding a couple of html files for multi-version functionality.

Tests and documentation

None needed

Acknowledgement

By contributing to this software project, I acknowledge that I have reviewed the software quality assurance guidelines and that my contributions are submitted under the Revised BSD License.

@kbonney
Copy link
Collaborator Author

kbonney commented Jul 20, 2023

Details

Right now the workflow doesn't work perfectly. I have tried to incorporate multiple versions of the docs, but certain limitations of GH actions makes it difficult to build multi-versions through an action rather than building it by hand and pushing it to main or some documentation branch. @kaklise We should discuss which versions/tags we actually want to be available on the documentation before I troubleshoot anymore. If we want multiple branches to be built and deployed we may have to create a separate documentation branch, but if its just tags I think we can keep things how they are (no dedicated docs branch).

Whatever we settle on, once we pull this in we will need to activate GH pages in the repo settings.

@coveralls
Copy link

coveralls commented Jul 20, 2023

Coverage Status

coverage: 84.417%. remained the same when pulling e5032d7 on kbonney:pages into 94b0e87 on USEPA:main.


on:
push:
branches: [main, pages]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we need a pages branch?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ah, no we don't. That's a relic from getting things to work on my fork without pushing into main. I'll remove

@kaklise kaklise merged commit 423e8cd into USEPA:main Aug 18, 2023
40 checks passed
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.

None yet

3 participants