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
Break up section 6 a bit more to make it more readable. #398
Break up section 6 a bit more to make it more readable. #398
Conversation
@dwsinger review required |
@nigelmegitt any feedback on this PR? |
@silviapfeiffer that's strange I'm sure I've looked at this and commented... Anyway, it's a big improvement, thank you. Reminder to self: section 5 Default Classes got added, so section 6 when I raised the issue has become section 7 in 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.
The new structure is a big improvement over the previous. The GitHub web based diff makes it hard to verify there's no substantive change, so I'm "approving" the structural change without checking that part of it.
@nigelmegitt I didn't introduce fundamental text changes - most of it is just moved text to clarify things a bit. |
Closes #397