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

Detailed recipe description #8

Open
kohlerm opened this issue Aug 19, 2013 · 13 comments
Open

Detailed recipe description #8

kohlerm opened this issue Aug 19, 2013 · 13 comments

Comments

@kohlerm
Copy link

kohlerm commented Aug 19, 2013

I'm relatively new to yard, and yard-chef, so please forgive me if this is a stupid question.

I understand that the brief description for the recipse comes from the metadata.rb file. I've got this working, but I wonder whether there is a way to also get a more detailed description, which would be documented within the recipe file probably at the top of the file.

Is this somehow possible?

@nitinmohan87
Copy link

@kohlerm - This is actually a very good question. The reason we did not read the description from the recipe file is because of how yard works. As far as I know, yard reads the comments that are attached to a specific keyword (like 'define', 'attribute', 'action', etc.). A recipe does not have a specific keyword (other than in metadata.rb) which we use in the recipe file. Hence any comments within the recipe file will be parsed as if they were part of its source code.

However, we can implement a custom yard tag (for example, @recipe) that can be defined at the top of the recipe file along with the description (for example, @recipe Description for the recipe). We can parse this tag and grab the description for the recipe.

Let me investigate more on the capabilities of yard for what else we can do. But your request is definitely possible.

@kohlerm
Copy link
Author

kohlerm commented Aug 20, 2013

Thanks for the quick reply!
Yes, I think your are correct, that was also what I learned yesterday about yard. Only extra-files would be scanned in any case, but those have to be specificied explicitly. I don't think that makes sense for recipe files.

I like your proposal, I will ask on the yard mailing list for advise.

Oh and thanks for inventing yard-chef!

Regards,
Markus

@nitinmohan87
Copy link

I looked at the issue you posted to yard. We are pretty much doing what the yard owner suggests for recipes even now. We read all files inside the recipe/ folder in the cookbook and put them into the source code for the recipes. I think having a custom yard tag would make it easier for us to parse the description, otherwise the description might get lost among the other comments in the recipes.

Thanks a lot for your encouragement on this project. Means a lot to us!

@kohlerm
Copy link
Author

kohlerm commented Aug 21, 2013

You are welcome! I would be trying to help it implemented if I would know a little bit more about yard ...

@nitinmohan87
Copy link

I can understand. We will be implementing this fix in our next dev cycle which should be beginning of next month.

@kohlerm
Copy link
Author

kohlerm commented Aug 22, 2013

Great!
Thanks
Markus

@JeanMertz
Copy link

@nitinmohan87 any progress on this? Would be nice to have this feature to properly document big recipes. By the way @kohlerm, as a workaround, you can always dynamically include a separate file in the metadata.rb file like so:

recipe 'base-system::default', IO.read(File.join(Dir.pwd, 'recipes', 'default.md'))

@nitinmohan87
Copy link

I apologize for the delay in coming up with this feature. We were held with higher priority stories that we never had the time to get to this. Kind of a lame excuse but that's what it is. I will try to get this done during my off time. But any contributions greatly appreciated.

@kohlerm
Copy link
Author

kohlerm commented Dec 13, 2013

Hi,
I missed that you updated this, sorry! Should this work now? I tried to use the recipe tag but it did not work for me.
I installed the gem,which is maybe not updated.

@tehZeno
Copy link

tehZeno commented Aug 21, 2014

Should this work as of today? (does not seem to work for me)

@nitinmohan87
Copy link

Sorry if that commit caused confusion. This feature is not yet implemented and I don't have a specific timeline for when it would available. Any contributions are appreciated!

@tehZeno
Copy link

tehZeno commented Aug 21, 2014

Ah yes. Thanks for the quick response.
The commit was a bit confusing.

Very busy on a project currently. But If no-one else picks it up in the
coming 3 months I'll have a look to see what I can contribute!

On Thu, Aug 21, 2014 at 8:52 PM, Nitin notifications@github.com wrote:

Sorry if that commit caused confusion. This feature is not yet implemented
and I don't have a specific timeline for when it would available. Any
contributions are appreciated!


Reply to this email directly or view it on GitHub
#8 (comment).

Franz-Robert van Vugt
| Email | Zeno@Zeno.Nu |
| Phone | 06 109 233 90 |
| Skype | Franz.Robert.Van.Vugt |

rshade pushed a commit that referenced this issue May 9, 2017
@rshade
Copy link

rshade commented May 9, 2017

can you try this with 2.0?

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

No branches or pull requests

5 participants