Release v1.0.0 #1253
Merged
Release v1.0.0 #1253
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This improves packaging reliability because it prevents tests from running against the current directory instead of the installed version of the package.
This required a large number of adjustments. I took the opportunity to make some further clean-up of top-matter.
Also any Python 2 clean-up that was handy.
* encoding header is no longer needed * from __future__ imports no longer needed * make module docstrings PEP 257 compliant.
Sphinx handling is actually better, not only using italic but also a mono-width font.
Done by `docformatter`, it can't get the summary line right if it's too long, but adjusts the description block width and gets the shorter docstrings right, so a lot better than doing them all by hand. The rest will have to wait for hand curation as we go.
Back in the day I thought it better to extract a test-specific fixture for each test. That had the benefit of making parameterized tests fold cleanly without the parameters in the decorator. The downside was you had to look in two places to understand the test so I dropped that practice. Fix some of these I need to change or add to later.
The old @lazyproperty implementation worked, but was much messier than the modern version, adding a member to the object __dict__. This later version that stores the value in the descriptor is much more tidy and more performant too I expect.
Also for example `BaseStoryPart` -> `StoryPart` for the same reason.
This removes some import cycles.
Metaclasses are different and somewhat better in Python 3. Modernize the `xmlchemy` custom element metaclass mechanism and remove Python 2 work-arounds that are no longer required.
Add additional run inner-content elements having a text equivalent, in particular `w:noBreakHyphen` and `w:ptab`. Give each of them their own custom element class having a `__str__()` method so they can each report their text content (constant in some cases like "-" for no-break hyphen).
Replace meta-programming with `enum.Enum` features built into Python 3 to support XML attribute mapping. Also some random typing and docstring clean-up along the way.
* clean up section (more needs doing in feature/steps and tests) * acceptance test * unit test
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Runinner-content, in particularRenderedPageBreak. andRun.iter_inner_content()to allow document-order processing ofRuncontent.Paragraphinner-content, in particularHyperlink. andParagraph.iter_inner_content()to allow document-order processing ofParagraphcontent.Paragraph.textincludes hyperlink text.