-
Notifications
You must be signed in to change notification settings - Fork 25
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
Refactor template helpers #321
Comments
I started working a bit on https://github.com/nimblehq/rails-templates/pull/318/files |
Please take this into account #317 (comment) |
Please take care of These Rubocop todos for |
@rosle I think @sanG-github will soon work on the prompt flow which should also refactor some of the helpers, should you two have a small call about sharing what was intended in this issue? Then could you let me know if this issue should be closed or reviewed? 👀 Many thanks 🙏 |
Why
Extract any helpers to separate modules or classes rather than putting them all in a template file.
Who Benefits?
Developers who work on the template
The text was updated successfully, but these errors were encountered: