terrapower / armi

An open-source nuclear reactor analysis automation framework that helps design teams increase efficiency and quality
https://terrapower.github.io/armi/
Apache License 2.0
214 stars 82 forks source link

Using ruff to enforce that no line can be longer than 140 characters #1637

Closed john-science closed 5 months ago

john-science commented 5 months ago

What is the change?

This PR uses ruff to enforce that no line can be longer than 140 characters. (Check out the changes to the pyproject.toml file.)

Why is the change being made?

I would like to get ARMI down to 100 characters. But that will change about 1000 files (though mostly in minor ways). So, that's a big step.

I consider this PR a first step in that direction. I have also looked at making a next PR after this one, that reduces the maximum line length to 120 characters, that only touches ~100 files. I think that PR will follow this one.

There are no code changes in this PR, just formatting changes.


Checklist

opotowsky commented 5 months ago

Ok we got the change in I wanted for the internal release so you can merge this one whenever you want!