Skip to content
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

Config split recipe #54

Open
andycarlberg opened this issue Apr 16, 2024 · 2 comments
Open

Config split recipe #54

andycarlberg opened this issue Apr 16, 2024 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@andycarlberg
Copy link

Is your feature request related to a problem? Please describe.
The recommended settings will handle config splits but expect a certain naming convention. This is not documented well (yet). If you make your own splits but don't follow or aren't aware of this convention, the splits fail to apply correctly.

In BLT, this was handled via a recipe that would generate the split config and the matching config directories so you could be sure your splits matched what the settings expected.

Describe the solution you'd like
I would suggest that this package implement a Drush command or Composer command to generate the splits according to a recipe. This seems like the appropriate place to implement it because it is specific to this package. If I'm not using this package to manage settings, then I don't need to follow this convention.

At a minimum, I think this package needs to document the expected config split convention.

Describe alternatives you've considered
This was previously handled in BLT but that project is EOL. Documentation would probably suffice but something more automated would be better.

@andycarlberg andycarlberg added the enhancement New feature or request label Apr 16, 2024
@vishalkhode1 vishalkhode1 self-assigned this Apr 23, 2024
@vishalkhode1
Copy link
Contributor

Hi @andycarlberg
Can you please confirm, are you expecting that DRS should provide a recipes blt recipes:config:init:splits and/or other recipes as well (or a documentation on how to create a new plugin/drush command for the same) ?
I can create a new plugin/drush command and but it would be great, if you could help us in testing/reporting bugs (If any).

@andycarlberg
Copy link
Author

Yes, that's correct. The blt recipes:config:init:splits command is what I feel should be replicated. It would help developers setup their config splits in the way this project expects so that it can handle automatic loading.

I would be happy to help test and report bugs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants