Skip to content

Conversation

@ekohl
Copy link
Collaborator

@ekohl ekohl commented Jul 13, 2021

With this empty line between the commend and the define, puppet-strings doesn't consider the comment as a documentation string. That means all the documentation that is written doesn't show up in REFERENCE.md.

I can't regenerate REFERENCE.md due to a missing puppet-module-posix-default-r3.0 gem so somebody else has to do that.

It was found in https://github.com/puppetlabs/puppetlabs-apache/pull/2157/files#r663838497 and introduced in 7e233fe.

@ekohl ekohl requested a review from a team as a code owner July 13, 2021 11:26
@puppet-community-rangefinder
Copy link

apache::vhost is a type

Breaking changes to this file WILL impact these 128 modules (exact match):
Breaking changes to this file MAY impact these 34 modules (near match):

This module is declared in 175 of 576 indexed public Puppetfiles.


These results were generated with Rangefinder, a tool that helps predict the downstream impact of breaking changes to elements used in Puppet modules. You can run this on the command line to get a full report.

Exact matches are those that we can positively identify via namespace and the declaring modules' metadata. Non-namespaced items, such as Puppet 3.x functions, will always be reported as near matches only.

@ekohl
Copy link
Collaborator Author

ekohl commented Jul 13, 2021

For what it's worth: https://github.com/voxpupuli/puppet-lint-param-docs would have caught this.

With this empty line between the commend and the define, puppet-strings
doesn't consider the comment as a documentation string. That means all
the documentation that is written doesn't show up in REFERENCE.md.

Fixes: 7e233fe
@ekohl ekohl force-pushed the fix-reference-docs branch from 53682b3 to bbfdc93 Compare July 22, 2021 10:51
@ekohl
Copy link
Collaborator Author

ekohl commented Jul 22, 2021

I now regenerated REFERENCE.md.

@ekohl
Copy link
Collaborator Author

ekohl commented Jul 22, 2021

@sheenaajay could you take a look?

@bastelfreak bastelfreak merged commit 92cf8d8 into puppetlabs:main Jul 22, 2021
@ekohl ekohl deleted the fix-reference-docs branch July 22, 2021 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants