-
Notifications
You must be signed in to change notification settings - Fork 11
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
Reorganized cookbooks dirs to be more self-explanatory #71
Comments
I like the proposed naming scheme (1-external, 2-overrides, 3-projects). Makes sense. I agree with @darkoantic that "merge" is very difficult to understand. |
So you don't see the fact that attributes exist at 3 levels (default/normal/override) as being an issue? People will open up a role to see how they work, and they're going to see "overrides" as one of the few properties. Seems confusing, but if no one else thinks so, we can leave it as "override". I'm not partial to "merge" as the alternative, for what it's worth, if anyone can think of a more novel term :) /cc quick feedback from folks new to chef? @nerdcorenet, @mgifford, @sylus |
In order to offer an opinion I need to understand first what is being suggested... Why three directories? What are they meant to be as full English descriptions? Once I can see that I might be able to offer some naming ideas. :) |
Ah gotcha. This explanation is good practice for me:
So was wondering, as a new person just getting to know Chef, what the most logical arrangement and naming might be, to help someone grapple with what each cookbooks directory does. Now that I think about it, this might be the most logical, since we currently keeping overrides and custom cookbooks in the same cookbooks directory (
Oh my god. Wall of text. I suppose I just really want to structure this project in such a way that it's as approachable as possible, because it be quite complex and daunting :) |
I can see what you are trying to achieve, but I don't think it is necessary to introduce the fourth directory. If you think that using the word
|
👍 Thanks Darko :) |
Related: applicationsonline/librarian#102
Proposal:
Helpful? Suggestions for better naming? I used to use
cookbooks-merge
, because "overrides" are already a separate concept in Chef and I didn't want to overload the term, but at the time @darkoantic suggestedcookbooks-overrides
was better. Having second thoughts now.The text was updated successfully, but these errors were encountered: