Skip to content

Conversation

@paulromano
Copy link
Contributor

Description

This PR makes the final changes for a 0.14.0 release:

  • Adds release notes
  • Add some missing .. versionadded:: directives
  • Fixes a broken latexpdf build for our documentation
  • Adds a missing virtual destrutor on the BoundaryCondition class that was causing a warning with clang

Checklist

  • I have performed a self-review of my own code
  • I have run clang-format (version 15) on any C++ source files (if applicable)
  • I have followed the style guidelines for Python source files (if applicable)
  • I have made corresponding changes to the documentation (if applicable)
  • I have added tests that prove my fix is effective or that my feature works (if applicable)

Copy link
Contributor

@pshriwise pshriwise left a comment

Choose a reason for hiding this comment

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

One small suggestion for you to consider regarding additional info on the new ordering of data returned from Mesh.vertices/centroids in a release note. Otherwise, I'm happy w/ it! 🚀

Copy link
Contributor

@pshriwise pshriwise left a comment

Choose a reason for hiding this comment

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

🚀

@pshriwise pshriwise merged commit fa23301 into openmc-dev:develop Nov 6, 2023
@paulromano paulromano deleted the release-0.14.0 branch November 7, 2023 18:34
church89 pushed a commit to openmsr/openmc that referenced this pull request Jul 18, 2024
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