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

(PDOC-224) Handle --emit-json(-stdout) again #162

Merged
merged 1 commit into from
Mar 1, 2018

Conversation

ekohl
Copy link
Contributor

@ekohl ekohl commented Mar 1, 2018

No description provided.

@@ -119,6 +119,8 @@ def build_generate_options(options = nil, *yard_args)
else
raise RuntimeError, "Invalid format #{options[:format]}. Please select 'json' or 'markdown'."
end
elsif options[:emit_json] || options[:emit_json_stdout]
generate_options[:json] = true
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also need to add generate_options[:path] = options[:emit_json] if emit_json is used.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated

@eputnam
Copy link
Contributor

eputnam commented Mar 1, 2018

@ekohl ++

@eputnam eputnam merged commit e18217b into puppetlabs:master Mar 1, 2018
@eputnam eputnam added the bugfix label Mar 1, 2018
@ekohl ekohl deleted the fix-emit-output branch March 1, 2018 18:12
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