Bug #9927
Section on module docs is incomplete/inconsistent with other docs
| Status: | Closed | Start date: | 10/05/2011 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: | % Done: | 0% |
||
| Category: | style guide | |||
| Target version: | - | |||
| Keywords: | Affected URL: | |||
| Branch: | ||||
| Votes: | 0 |
Description
The Style Guide section on module documentation doesn’t match our own docs on puppet doc style, and appears to have at least a single line, maybe more, truncated from the top.
A snippet from the beginning of the Style Guide code:
# Full description of class here.
#
# == Parameters
#
# Document parameters here
#
# [*servers*]
# Description of servers class parameter. e.g. "Specify one or more
# upstream ntp servers as an array."
#
# == Variables
And from the wiki (http://projects.puppetlabs.com/projects/1/wiki/Puppet_Manifest_Documentation) – which is the only place I can find real docs on format:
# Class: users
#
# This class installs our default set of users in our servers
#
# Parameters:
# $starting_uid:
# this global variable is used to set the minimum uid used for our users
#
# Actions:
# Install the default set of users: [dana,fox]
#
For the record, the second matches my own usage and that which I’ve seen elsewhere.
History
Updated by James Turnbull 7 months ago
- Status changed from Unreviewed to Needs Decision
- Assignee set to Nick Fagerlund
Updated by Cody Herriges 5 months ago
- Status changed from Needs Decision to Rejected
The format put forth by the style guide is proper rdoc and was agreed upon. This is not reflected in all the modules we publish but the template we generate has been updated in puppet core.
https://github.com/puppetlabs/puppet/blob/599a146386965f56b015530da59f3dd7cfc12415/lib/puppet/module_tool/skeleton/templates/generator/manifests/init.pp.erb
Updated by Eric Shamow 5 months ago
- Status changed from Rejected to Re-opened
Sorry to re-open, but the issue is the mismatch. I have no preference for which format we use, but one of the two needs to be corrected.
Additionally the style guide is still apparently missing info – it seems that lines are missing from the top of the Style Guide section.
Updated by Nick Fagerlund 5 months ago
Hmm. Yeah, let’s make them match. I’ll fix the wiki page immediately and take a second look at whatever I put in Learning Puppet. Eric, could you make any changes the style guide needs (re: the missing lines) and submit a pull request?
Updated by Eric Shamow 5 months ago
- Status changed from Re-opened to Closed
Nick,
Style guide actually looks ok now…I may have been looking in the wrong place or it’s been updated since. But this looks resolved to me.
-Eric