diff options
author | Tim Smith <tsmith84@gmail.com> | 2020-10-28 14:08:47 -0700 |
---|---|---|
committer | Tim Smith <tsmith84@gmail.com> | 2020-10-28 15:44:11 -0700 |
commit | 8dbf0e23da28c3a869265331b04b33ceb9467ff0 (patch) | |
tree | 0b08ed2d577c5756e5299caa1e91efb73f9b61ab /tasks | |
parent | 79772c0ca81d6c7481f74aa07ae5a5873698cf24 (diff) | |
download | chef-8dbf0e23da28c3a869265331b04b33ceb9467ff0.tar.gz |
Correctly generate docs yaml files to include package warnings
This data actually needed to be under resource_description_list and not at the top level.
Signed-off-by: Tim Smith <tsmith@chef.io>
Diffstat (limited to 'tasks')
-rwxr-xr-x | tasks/docs.rb | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/tasks/docs.rb b/tasks/docs.rb index 5ad4d2baf4..f21a80b5b4 100755 --- a/tasks/docs.rb +++ b/tasks/docs.rb @@ -227,9 +227,6 @@ namespace :docs_site do properties["unit_file_verification"] = true if name == "systemd_unit" - # these packages all provide 'package' depending on OS and we want to inject a warning / note that you can just use 'package' instead - properties["notes_resource_based_on_package" ] = true if %w{apt_package bff_package dnf_package homebrew_package ips_package openbsd_package pacman_package portage_package smartos_package solaris_package windows_package yum_package zypper_package}.include?(name) - properties end @@ -237,7 +234,7 @@ namespace :docs_site do # using the markers "Note:" for "note" sections and "Warning:" for "warning" sections. # TODO: has the limitation that the plain description section is assumed to come first, # and is followed by one or more "note"s or "warning"s sections. - def build_description(text) + def build_description(name, text) return [{ "markdown" => nil }] if text.nil? description_pattern = /(Note:|Warning:)?((?:(?!Note:|Warning:).)*)/m @@ -262,6 +259,9 @@ namespace :docs_site do end end + # if we're on a package resource, depending on the OS we want to inject a warning / note that you can just use 'package' instead + description << { "notes_resource_based_on_package" => true } if %w{apt_package bff_package dnf_package homebrew_package ips_package openbsd_package pacman_package portage_package smartos_package solaris_package windows_package yum_package zypper_package}.include?(name) + description end @@ -281,7 +281,7 @@ namespace :docs_site do r["resource"] = name r["aliases"] = ["/resource_#{name}.html"] r["menu"] = build_menu_item(name) - r["resource_description_list"] = build_description(data["description"]) + r["resource_description_list"] = build_description(name, data["description"]) r["resource_new_in"] = data["introduced"] unless data["introduced"].nil? r["syntax_full_code_block"] = generate_resource_block(name, properties, data["default_action"]) r["syntax_properties_list"] = nil |