Ticket #10159 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Fix indentation errors in calibration modules

Reported by: Martyn Gigg Owned by: Peter Peterson
Priority: critical Milestone: Release 3.3
Component: Python Keywords:
Cc: Blocked By:
Blocking: Tester: Martyn Gigg

Description

Some Sphinx warnings have shown up in develop_clean relating to autodoc not being able to import the calibration module: http://builds.mantidproject.org/job/develop_clean/label=rhel6-build/299/consoleFull (search for calibration.rst)

Change History

comment:1 Changed 6 years ago by Martyn Gigg

  • Blocking 10160 added

comment:1 Changed 6 years ago by Martyn Gigg

  • Blocking 10160 removed

Note: #10160 has been created to implement tests for these modules.

comment:3 Changed 6 years ago by Pete Peterson

Re #10159. Fixing whitespace issue.

Changeset: 82393f7d4bc8f48766340a29c65e5e9492d9f6e1

comment:4 Changed 6 years ago by Pete Peterson

Re #10159. Fixing python whitespace issues.

Changeset: 4008508101a6219bc84d64dc50351069746750e3

comment:5 Changed 6 years ago by Pete Peterson

Re #10159. Fixing python whitespace issues.

Changeset: 7e5b8ea6cc9a236ae171b5b07514aa8ab60ec581

comment:6 Changed 6 years ago by Pete Peterson

Re #10159. Fixing python whitespace issues.

Changeset: 26c54dc566af77a7f0943ce91c546c5b5ef58929

comment:7 Changed 6 years ago by Pete Peterson

Re #10159. Fixing python whitespace issues.

Changeset: e49ad7d9e7f1838fdfce57350f2ade805b249e3c

comment:8 Changed 6 years ago by Pete Peterson

Re #10159. Fixing python whitespace issues.

Changeset: d6f4ed797f5c9c8fef6c3d566efd7d07379109f8

comment:9 Changed 6 years ago by Peter Peterson

  • Status changed from new to assigned

comment:10 Changed 6 years ago by Peter Peterson

  • Status changed from assigned to verify
  • Resolution set to fixed

This is on bugfix/10159_python_whitespace

To test: Notice that the errors in the develop_rhel7 build are no longer due to python.

comment:11 Changed 6 years ago by Martyn Gigg

  • Status changed from verify to reopened
  • Resolution fixed deleted

The clean develop builds are still picking up some Sphinx warnings in docstrings: http://builds.mantidproject.org/view/Develop%20Builds/job/develop_clean/label=win7-build/311/warnings3Result/. I suspect that they are all indentation issues.

comment:12 Changed 6 years ago by Peter Peterson

  • Status changed from reopened to verify
  • Resolution set to fixed

This is being processed in git pull#27.

comment:13 Changed 6 years ago by Pete Peterson

Re #10159. Hopefully fixing the sphinx builds.

Changeset: f1fcbd8eaa2d5c1a9c868a8ea8f135d8cfb6a363

comment:14 Changed 6 years ago by Martyn Gigg

  • Status changed from verify to verifying
  • Tester set to Martyn Gigg

comment:15 Changed 6 years ago by Martyn Gigg

  • Status changed from verifying to closed

comment:16 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 11001

Note: See TracTickets for help on using tickets.