Ticket #11785 (closed: fixed)
Fix pylint issues in EnginX algorithms
| Reported by: | Federico M Pouzols | Owned by: | Federico M Pouzols |
|---|---|---|---|
| Priority: | minor | Milestone: | Release 3.5 |
| Component: | Framework | Keywords: | Maintenance |
| Cc: | Blocked By: | ||
| Blocking: | Tester: | Dan Nixon |
Description (last modified by Federico M Pouzols) (diff)
There are several issues in these algorithms and aux files:
- EnginXFitPeaks: http://builds.mantidproject.org/job/pylint_master/lastBuild/violations/file/EnginXFitPeaks.py/
- EnginXCalibrateFull: http://builds.mantidproject.org/job/pylint_master/lastBuild/violations/file/EnginXCalibrateFull.py/
- EnginXCalibrateTest: http://builds.mantidproject.org/job/pylint_master/lastBuild/violations/file/EnginXCalibrateTest.py/
- EnginXUtils: http://builds.mantidproject.org/job/pylint_master/lastBuild/violations/file/EnginXUtils.py/
- EnginXCalibrate: http://builds.mantidproject.org/job/pylint_master/lastBuild/violations/file/EnginXCalibrate.py/
- EnginXFocus: http://builds.mantidproject.org/job/pylint_master/lastBuild/violations/file/EnginXFocus.py/
You can find the updated list of pylint issues/violations from http://builds.mantidproject.org/job/pylint_master/lastBuild/violations/
Change History
comment:2 Changed 5 years ago by Federico Montesino Pouzols
- Status changed from new to inprogress
try to address all pylint issues in EnginX* algorithms, re #11785
Changeset: 7b7cacd39e366fa36a7c23e48718f0140c045f06
comment:3 Changed 5 years ago by Federico Montesino Pouzols
- Status changed from inprogress to verify
- Resolution set to fixed
This is being verified as pull request #771.
comment:4 Changed 5 years ago by Federico Montesino Pouzols
don't use super as this is an old-style class, re #11785
Changeset: 31b76dbdfeb5f25f89872bee4947f0f8e2ec863f
comment:5 Changed 5 years ago by Dan Nixon
- Status changed from verify to verifying
- Tester set to Dan Nixon
comment:6 Changed 5 years ago by Federico Montesino Pouzols
try to address all pylint issues in EnginX* algorithms, re #11785
Changeset: 7b7cacd39e366fa36a7c23e48718f0140c045f06
comment:7 Changed 5 years ago by Federico Montesino Pouzols
don't use super as this is an old-style class, re #11785
Changeset: 31b76dbdfeb5f25f89872bee4947f0f8e2ec863f
comment:8 Changed 5 years ago by Dan Nixon
- Status changed from verifying to closed
Merge pull request #771 from mantidproject/11785_fix_pylint_issues_enginx_algorithms
Fix Pylint issues in EnginX* algorithms
Full changeset: 6bdc7feeaec3bffc29bb02d8013f9232fa345349
comment:10 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 12623
