Skip to content

Commit

Permalink
ARMI has always had super statements (#1155)
Browse files Browse the repository at this point in the history
  • Loading branch information
john-science committed Feb 8, 2023
1 parent 4fa97fa commit d53c17d
Showing 1 changed file with 2 additions and 6 deletions.
8 changes: 2 additions & 6 deletions doc/developer/standards_and_practices.rst
Original file line number Diff line number Diff line change
Expand Up @@ -302,9 +302,5 @@ General do's and don'ts
do not use ``print``
ARMI code should not use the ``print`` function; use one of the methods within ``armi.runLog``.

do not use ``super``
In preference to the ``super`` function, explicitly call the parent object's method. For example, in an
``__init__``, use ``ParentClass.__init__(self, plus, additional, arguments)``.

do not leave ``TODO`` statements in production code
If your ``TODO`` statement is important, perhaps it should be a GitHub Issue.
Do not add new ``TODO`` statements in your commits and PRs.
If your new ``TODO`` statement is important, it should be a GitHub Issue. Yes, we have existing ``TODO`` statements in the code, those are relic and should be handled.

0 comments on commit d53c17d

Please sign in to comment.