Skip to content

Update custom renderers to overload render#254

Open
rsamoilov wants to merge 1 commit intomainfrom
custom-renderers-update
Open

Update custom renderers to overload render#254
rsamoilov wants to merge 1 commit intomainfrom
custom-renderers-update

Conversation

@rsamoilov
Copy link
Copy Markdown
Member

This pull request introduces a redesign of the custom renderer system. Instead of generating separate render_<name> methods for each custom renderer, the system now overloads the main render method in controllers to support custom renderers via keyword arguments.

ref #244

@rsamoilov
Copy link
Copy Markdown
Member Author

Hey @anuj-pal27

I've been thinking about the render_* approach more, and after an extensive grilling session with Gemini I decided to update your implementation to overload the render method instead. The main argument is discoverability - seeing render something: true leads the developer to check the documentation for render and discover a link to the custom renderers functionality. Seeing render_something doesn't tell a developer not familiar with the framework anything.

Would love your review!

@rsamoilov rsamoilov force-pushed the custom-renderers-update branch from 085f4f3 to 7805151 Compare March 31, 2026 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant