• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

terrapower / armi / 867
87%

Build:
DEFAULT BRANCH: master
Ran 13 May 2021 02:08PM UTC
Jobs 1
Files 396
Run time 6min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

pending completion
867

push

travis-ci

web-flow
Another Pylint PR (#294)

* Removing useless pylint code duplication detection

* Removing circular imports

I found some circular imports all over the code base most stemming from this line:

import armi

So, lots of our base code still has this, as it will take significant redesign of
the global package variables and setting stuff. However, some of the instances
were easy to fix. Particularly the unit tests.

I should note that Python does a good job of avoiding circular imports, so most
of these weren't a REAL problem. However, it is probably bad practice to use
import armi in our code base, basically anywhere.

* Pylint cleanup - C0113 correct use of not in

* Pylint cleanup - R1714 consider-using-in

* Pylint cleanup - W0612 unused-variable

* py2to3 - removing future print import

Several files still had this line at the top:
from __future__ import print_function
Generally, this code is only useful in Python v2.

There are only a tiny amount of edge cases where
this line is useful in Python v3, mostly having
to do with trying to connect to Python v2 modules.
But it doesn't appear to me this is the case.

* pylint cleanup - W0612 unused-variable

* Pylint cleanup - R1711 useless-return

* Addressing PR comments

Mostly, this commit addresses the renaming/removing of unused variables.
But this also addresses some conflicts between the branches.

* Filling out unit tests

Here I have added unit tests to the materials area.
These tests are not creative or original, and where possible I have tried
to verify the numbers using various sources online.
One reason to add unit tests to this PR is to increase the code coverage
of the PR as a whole, which dropped due to removing lines of code.

194 of 194 new or added lines in 49 files covered. (100.0%)

32543 of 40143 relevant lines covered (81.07%)

0.81 hits per line

Jobs
ID Job ID Ran Files Coverage
3 867.3 (TOXENV=cov,report) 13 May 2021 02:08PM UTC 0
81.07
Travis Job 867.3
Source Files on build 867
Detailed source file information is not available for this build.
  • Back to Repo
  • Travis Build #867
  • 260f518f on github
  • Prev Build on master (#862)
  • Next Build on master (#869)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc