-
Notifications
You must be signed in to change notification settings - Fork 48
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
2to3 tutorial #2310
2to3 tutorial #2310
Conversation
3ecbbec
to
fcdca7e
Compare
fcdca7e
to
181e972
Compare
Tests passing, good to go. |
1a96eeb
to
0849433
Compare
disable currently un-runnable tests: * rose suite-run * gtk related code
0849433
to
7454caa
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've made some comments, but none of them are fatal objections to this PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great!
- Docs still build cleanly;
check-software
still works, with new appropriate section & my installedpillow
library found;- The tweaked tutorial suites making use of the Relative ICP feature (under Python 3 via the Cylc 8
master
branch) run as they should, & have the correct ICP & FCP; - The relevant docs sections still make sense by eyeball with the new ICP-setting method.
This sets the :term:`final cycle point` six hours after the | ||
:term:`initial cycle point`. | ||
|
||
Run `cylc validate` to check for any errors:: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Run `cylc validate` to check for any errors:: | |
Run ``cylc validate`` to check for any errors:: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good spot! I'll try to note this for a future fix. I wish there was a way to get Sphinx to raise errors for things like this. We might be able to write a custom role or something of that ilk...
Convert the Rose/Cylc tutorial suites to Python3
png.py
, replace with pillow (new dependency).The related test is
t/docs/02-tutorial-suites.t
which should pass with three skips.