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

Fixes #31215 - Generate DSL docs #892

Merged
merged 1 commit into from Oct 29, 2020

Conversation

ofedoren
Copy link
Member

@ekohl, please, let me know if I should link it to the Redmine or add/change something else.

My intention here is to add generating of DSL docs similarly to API docs after Foreman or a plugin is installed.

manifests/database.pp Outdated Show resolved Hide resolved
@ofedoren
Copy link
Member Author

@ekohl, what's the status here?..

Copy link
Contributor

@wbclark wbclark left a comment

Choose a reason for hiding this comment

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

This change worked fine for me both in testing the individual puppet module and also in foreman-installer (tested via Forklift with the module_prs feature)

@ofedoren before we merge this could you please create a Redmine Issue and update the commit message and PR title to reference the Issue # ? For example, "Fixes #XXXXX - Generate DSL docs"

@ofedoren ofedoren changed the title Add DSL docs generating Fixes #31215 - Generate DSL docs Oct 29, 2020
@ofedoren
Copy link
Member Author

Thanks, @wbclark! Done.

@wbclark wbclark merged commit a879e59 into theforeman:master Oct 29, 2020
@wbclark
Copy link
Contributor

wbclark commented Oct 29, 2020

Thanks @ofedoren ! It is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants