search

geraintpalmer / DynamicClasses

MIT License
1 stars 0 forks source link

Review of draft #21

Closed drvinceknight closed 1 week ago

drvinceknight commented 1 month ago

There's one main restructure suggestion I have and then I'd like to take another look (if you'd like me to). Happy to chat about any of the above (or for any of the above to be ignored :)).

geraintpalmer commented 1 week ago

Thanks @drvinceknight

I've now implemented these changes and done the restructure. It's all on the main branch.

drvinceknight commented 1 week ago

Cool. Let me know if you'd like me to take another look.

Edit: Sorry got confused, should have said "I'll take another look" :) Should do this tomorrow.