Amend Proposed Classes - #235
Draft
Jediweirdo wants to merge 5 commits into
Draft
Conversation
Initial commit of my 5 ideas to extend class usfulness. Many are not scoped at the moment. Let me know your thoughts
Some rendering/formatting issues and small inconsistencoes became apparent to me after reading.
The master example in the summary section was never properly updated alongside the changes made to the design terminology, so that was done. I also realized that the exact semantics of the fields needed more explanation, so I provided it. Finally, more janitor things were done because the drawbacks/alteritives sections had to up reformatted or otherwise cleaned up due to shifting context ruining some things
small quick fix
Some of the comments still weren't updated following the design changes made throughout the course of creating the first draft. It's fixed now
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Rendered. Note that if this ever leaves a draft state (which it may not), this will likely be split into multiple PRs. For now, I think it would be easier to dump all of my thoughts on classes in 1 place. Let me know if you would prefer breaking up some of these ideas in the future