Page MenuHome

info_overview.rst doc cleanup
ClosedPublic

Authored by nBurn (nBurn) on Jun 2 2017, 1:41 AM.

Diff Detail

Repository
rB Blender

Event Timeline

nBurn (nBurn) created this revision.Jun 2 2017, 1:41 AM

Looks good only a few small nick picky things.

doc/python_api/rst/info_overview.rst
21

Should --> can be

22

Maybe document the other modules that are built in by default?

23

"Blender data, classes, and functions." --> Blender's data, classes, and functions.

26

Should be "an object" instead of the object.

57

"the classes the script" --> "the classes that the script"

62

"by importing that module." --> "by importing that module again."

427

Use a colon here instead of ...

This revision is now accepted and ready to land.Jun 2 2017, 3:43 AM

I mostly agree with all your suggestions, I left a comment for the others.

doc/python_api/rst/info_overview.rst
21

I was trying to imply that not all Python code will run as it would with the standard Python runtime (eg: Tkinter). This may not be worth going into though.

22

I left this part mostly the same as in the original as I was confused about what the original wording was referring to.

23

Agree.

26

Agree.

57

Agree.

62

This was another part where I was uncertain what the original document was describing and decided to err on the side of caution and leave it mostly unchanged.

427

Agree.

doc/python_api/rst/info_overview.rst
21

ok.

22

Actually I think adding :mod:mathutils` `would be enough

62

I think it is fine to just add "again" to the end of the sentence.

Updated diff patch with Blendify's recommendations.

nBurn (nBurn) marked 14 inline comments as done.Jun 2 2017, 8:56 PM
This revision was automatically updated to reflect the committed changes.