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

URL & documentation updates for 3.1 release #1807

Merged
merged 5 commits into from
Nov 9, 2024

Conversation

speth
Copy link
Member

@speth speth commented Nov 7, 2024

Changes proposed in this pull request

  • Update URLs to correspond to the new website organization coming with the Cantera 3.1 release. This PR handles those URLs that can be updated now, for links that are now internal within the version-specific documentation or are to other fully qualified URLs. A separate PR will handle those updates that can only occur simultaneous with the deployment of the new website and the 3.1 release.
  • Add a documentation page covering the steps needed to make a new Cantera release (adapted/updated from the cantera-release-guide repo)
  • Fix some typos in the documentation

If applicable, fill in the issue number this pull request is fixing

Partially resolves #1748.

Checklist

  • The pull request includes a clear description of this code change
  • Commit messages have short titles and reference relevant issues
  • Build passes (scons build & scons test) and unit tests address code coverage
  • Style & formatting of contributed code follows contributing guidelines
  • The pull request is ready for review

@speth speth marked this pull request as ready for review November 8, 2024 03:54
Copy link
Member

@ischoegl ischoegl left a comment

Choose a reason for hiding this comment

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

Thanks @speth. I can't comment on the release instructions, but the rest looks good to me!

Copy link
Member

@bryanwweber bryanwweber left a comment

Choose a reason for hiding this comment

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

Thanks Ray!

@ischoegl ischoegl merged commit ce858af into Cantera:main Nov 9, 2024
52 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.

Fix links referencing old website structure
3 participants