Closed BrannonKing closed 6 years ago
Seems like there is a mix of py2 and py3 exclusive syntax used (print
keyword in contrib/devtools/clang-format.py
which will fail in py3 as syntax error and def get_obj_from_dirty_text(full_object: str):
which will fail as syntax error in py2).
Is this intentional? Should all code be only py2, py3 or should it work in both py2 and py3?
@eukreign , the clang-format scripts predate my time here. I just made it run as it didn't before. I don't know that anyone here wrote them, judging from the copyright messages in them. As far as I know, they have always been Py2 scripts. For this PR, the focus should be on the "docs" scripts.
Alright, it was in this PR so I thought it was related. Which files should I review as part of this PR?
@BrannonKing can this be merged? also as part of this, can you run the script (the one that matches the daemon script) and commit the output as well?
Open questions on this:
We have a request to make the output available on the web.
Following from my answer on slack about how to unit test these scripts: to be able to use the generated output as the test itself, you have to commit and version the output deliberately (not via travis) so that when the docs do change it can be verified for correctness (did the docs change because the input source changed or did they change because the generator script changed?). Therefore, my advice would be:
Looks good to me. Has this been run through clang-format? It seems travis is only unhappy with style.
I have not ran the formatting. I'm having to undo 20 years of habit to remember to format before checking in. I still want to make that a report rather than a requirement.
It sounds like we would additionally like a report from Travis (or the local build) that states that the generated API doc needs to be updated. This involves starting the server. Have we done that on Travis before? Do we run any integration tests on Travis?
Do we run any integration tests on Travis?
No. Possibly related to #164
Issue: #131 . There are two scripts: