add BaseRobyn class

[dave42w]

Description

This PR refactors Robyn and SubRouter by adding an abstract BaseRobyn that is a common superclass.

So far the only method in Robyn itself (and therefore not in SubRouter) is start.

PR Checklist

Please ensure that:

• The PR contains a descriptive title
• The PR contains a descriptive summary of the changes
• You build and test your changes before submitting a PR.
• You have added relevant documentation
• You have added relevant tests. We prefer integration tests wherever possible

Pre-Commit Instructions:

• Ensure that you have run the pre-commit hooks on your PR.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
robyn	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Dec 31, 2024 7:59am

[codspeed-hq]

CodSpeed Performance Report

Merging #1100 will not alter performance

_{Comparing dave42w:BaseRobyn (6b3f6bd) with main (2aad774)}

Summary

✅ 138 untouched benchmarks

[dave42w]

@sansyrox two questions about this

1. Are you happier about keeping all this in init.py or do you want me to move BaseRobyn into baserobyn.py?
2. Which other methods do you think are ready to move out of BaseRobyn into Robyn (ie that SubRouter should not use).

Thanks

[sansyrox]

Hey @dave42w 👋

Thanks for the PR 😄

1. Let's keep everything in __init__.py for now.
2. Let me do a complete review and get back 😄

[sansyrox]

Hey @dave42w 👋

Overall, this is a great start point for the refactor. Looks good to me!! ❇️

[sansyrox]

we probably also don't need a file_path in the subrouters now, right?

[dave42w]

we probably also don't need a file_path in the subrouters now, right?

Yup the init was going to be the next step.

[sansyrox]

Good job and thanks @dave42w !