-
Notifications
You must be signed in to change notification settings - Fork 40
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
Documentation Update #591
Documentation Update #591
Conversation
Build Status Report - 95adfa1 - 2024-03-25 22:48:08 +0000Build
|
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.
Thanks @bennibbelink - this looks good. Do we merge ASAP, or do we need to do the release first/simultaneously? I ask because the (working/up-to-date) DEB packages don't actually exist yet, do they?
Good point, we should probably wait until the release. |
It does create a bit of a race condition... since it would be good to update docs as much as possible before release :) |
I wonder if maybe in the cyclus.github.com #349 we should use the Up to you whether we merge this PR now or wait. The DEB package links are broken now anyway - changing the broken links for different broken links doesn't sound like the end of the world |
Thanks @bennibbelink - let's do this now and revisit what we need to at release time |
The biggest changes are
master
tomain
. I also updated the Ubuntu versions and changed the links to point to our GitHub releases.The minimum required Cyclus version will need to be updated again when we issue the new release.
I will make a separate PR for documenting our container images as an installation option.