Skip to content
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

Rationalize Perseus idioms #32

Closed
arctic-hen7 opened this issue Sep 28, 2021 · 1 comment · Fixed by #37
Closed

Rationalize Perseus idioms #32

arctic-hen7 opened this issue Sep 28, 2021 · 1 comment · Fixed by #37
Assignees
Labels
A-templates Area: templates C-docs Category: documentation D-easy Difficulty: easy P-medium Priority: medium S-in-development Status: in development
Milestone

Comments

@arctic-hen7
Copy link
Member

Is your feature request related to a problem? Please describe.
If you take three seconds to go through the Perseus idiom of defining functions that return closures within Rcs, you'll probably stop liking it pretty quickly, because it's grotesque. Besides, we can trivially just have those closures as functions themselves that are then placed in Rcs (which is also an idiom, but only for some things...).

Describe the solution you'd like
In short, these idioms need to start making sense! A lot of them are left over from the old architecture of v0.1.0 and even before that, and they should be updated in the documentation and examples to reflect better practice (which is more concise and logical).

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
This is primarily a documentation change, but it's a very big one, and it'll be released in v0.3.0.

@arctic-hen7 arctic-hen7 added C-docs Category: documentation A-templates Area: templates D-easy Difficulty: easy P-medium Priority: medium S-in-development Status: in development labels Sep 28, 2021
@arctic-hen7 arctic-hen7 self-assigned this Sep 28, 2021
@arctic-hen7 arctic-hen7 added this to the v0.3.0 milestone Sep 28, 2021
@arctic-hen7
Copy link
Member Author

I'll work on this with the changes in #31.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-templates Area: templates C-docs Category: documentation D-easy Difficulty: easy P-medium Priority: medium S-in-development Status: in development
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant