Skip to content

Support the glob pattern in root_dir and model_dir #198

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 3 commits into from
May 30, 2025

Conversation

sinsoku
Copy link
Contributor

@sinsoku sinsoku commented Feb 27, 2025

We often want to specify multiple directories in root_dir and model_dir.
For example, when using engines, packs-rails, etc.

This commit supports the glob pattern with these options, making it easy
to specify multiple directories.

fixes #99

We often want to specify multiple directories in `root_dir` and `model_dir`.
For example, when using engines, packs-rails, etc.

* https://github.com/rubyatscale/packs-rails
* https://guides.rubyonrails.org/engines.html

This commit supports the glob pattern with these options, making it easy
to specify multiple directories.

fixes drwl#99
@sinsoku sinsoku force-pushed the support-glob-pattern branch from f6d4773 to 08f527c Compare March 3, 2025 06:50
@sinsoku
Copy link
Contributor Author

sinsoku commented Mar 3, 2025

@drwl
CI failed, so I fixed Style/StringLiterals and force-pushed.

@drwl
Copy link
Owner

drwl commented Apr 5, 2025

@sinsoku just checking -- have you been able to try these changes in your local? I don't have a Rails app with packwerk setup to test.

Copy link
Owner

@drwl drwl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes lgtm -- waiting on confirmation of manual testing before merging

@sinsoku
Copy link
Contributor Author

sinsoku commented Apr 14, 2025

@drwl I have tested it in a Rails app using packwerk.
It worked with this change.

@jarredhawkins
Copy link
Contributor

Also successfully tested this in a large Packwerked application. Thank you both for the contributions here!

@drwl
Copy link
Owner

drwl commented May 30, 2025

Thanks for the confirmations @sinsoku and @jarredhawkins .

@drwl drwl merged commit 98d7856 into drwl:main May 30, 2025
22 checks passed
@sinsoku sinsoku deleted the support-glob-pattern branch May 31, 2025 09:10
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.

Feature request: packs-rails support
3 participants