Closed njzjz closed 4 months ago
[!IMPORTANT]
Auto Review Skipped
Review was skipped as selected files did not have any reviewable changes.
Files selected but had no reviewable changes (1)
* reacnetgenerator/py.typedYou can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
The recent project updates involve enhancing type safety, readability, and compatibility. Changes include upgrading Pyright in the workflow, adding return type annotations in _detect.py
, specifying types in _matrix.py
, and removing type: ignore
annotations in _path.py
.
File | Change Summary |
---|---|
.github/workflows/pyright.yml |
Updated Pyright version from 1.1.308 to 1.1.363 . |
reacnetgenerator/_detect.py |
Added return type annotations to _readNfunc and _readstepfunc methods in _Detect and _DetectLAMMPSbond classes. |
reacnetgenerator/_matrix.py |
Modified _printtable to include species_idx variable declaration and specify index and columns explicitly in pd.DataFrame call. |
reacnetgenerator/_path.py |
Removed type: ignore annotations from convertSMILES method for RDKit-related calls. |
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Comparing njzjz:pyright
(323ade0) with master
(311db70)
✅ 8
untouched benchmarks
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 95.18%. Comparing base (
311db70
) to head (323ade0
). Report is 13 commits behind head on master.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Summary by CodeRabbit
Chores
1.1.308
to1.1.363
.Refactor
_detect.py
._matrix.py
.Bug Fixes
convertSMILES
method of thePath
class in_path.py
.