8000 Convert Part generator to use RubyFileWriter and Generate::Command by cllns · Pull Request #292 · hanami/cli · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Convert Part generator to use RubyFileWriter and Generate::Command #292

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

Merged
merged 55 commits into from
Apr 18, 2025

Conversation

cllns
Copy link
Member
@cllns cllns commented Feb 28, 2025

Builds on #288

After this is merged, I want to refactor some of this. I don't love the names relative_parent_class (gets the app/slice namespace prepended) and absolute_parent_class. Considering using qualified/unqualified but that's not super clear either. Any ideas? parent_class and parent_class_within_namespace?

In that PR, I'll also go through and change all references to RubyFileWriter to RubyClassFile, since that's the API we want to use going forward.

@cllns cllns requested a review from timriley February 28, 2025 21:16
Base automatically changed from expand-use-of-ruby-file-writer-to-action to main April 18, 2025 21:36
@cllns cllns merged commit 17699d1 into main Apr 18, 2025
4 checks passed
@cllns cllns deleted the expand-use-of-ruby-file-writer-to-view-part branch April 18, 2025 21:44
cllns added a commit that referenced this pull request Apr 18, 2025
cllns added a commit that referenced this pull request Apr 24, 2025
* Rename to partially_qualified_parent

* Rename to fully_qualified_parent

* Convert Operation generator to use RubyClassFile

* Convert Relation generator to use RubyClassFile

* Convert Repo generator to use RubyClassFile

* Convert Struct generator to use RubyClassFile

* Convert Repo generator to use RubyClassFile

* Convert Component generator to use RubyClassFile

* Remove RubyFileWriter, in favor of RubyClassFile

* Remove un-used ERB files

Clean up from #292
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
0