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

Removing long value of static member in class of test_composities fro… #1474

Merged
merged 1 commit into from
Nov 13, 2023

Conversation

zachmprince
Copy link
Contributor

@zachmprince zachmprince commented Nov 13, 2023

What is the change?

This PR adds a doco meta field in reactor.tests.test_composities.MockBP that tells sphinx to not display the value of a couple static members.

Why is the change being made?

These members were essentially listing every nuclide in our library, which isn't very useful from a documentation point of view. It was also preventing a PDF build of the documentation.


Checklist

  • The release notes (location doc/release/0.X.rst) are up-to-date with any important changes.
  • The documentation is still up-to-date in the doc folder.
  • No requirements were altered.
  • The dependencies are still up-to-date in pyproject.toml.

@CLAassistant
Copy link

CLAassistant commented Nov 13, 2023

CLA assistant check
All committers have signed the CLA.

@john-science john-science self-requested a review November 13, 2023 22:50
@john-science john-science added the documentation Improvements or additions to documentation label Nov 13, 2023
@john-science john-science merged commit 6603c1d into terrapower:main Nov 13, 2023
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants