-
Notifications
You must be signed in to change notification settings - Fork 85
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
Remove steps from generating content #3017
Remove steps from generating content #3017
Conversation
The PR preview for 086a348 is available at theforeman-foreman-documentation-preview-pr-3017.surge.sh The following output files are affected by this PR: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Weren't you supposed to move it to another module? Also, your Jira issue is requesting further improvements. Where are those?
I think the instructions for syncing
The customers are recommended to do import syncable same way as the default one via hammer. |
I'll revert the snippet back to its original file and then update here. That snippet works across other areas of documentation so I shouldn't have done that. |
It was decided that certain steps in generating content are not necessary due to conflicting statements about importability of syncable exports.
Snip file restored and info from snip file that needed to be removed specifically for this section to avoid contradictory statements was put in the section.
d5d500c
to
0493638
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple of nitpicks.
The snippet itself could probably use improvement.
guides/common/modules/proc_exporting-the-library-environment-in-a-syncable-format.adoc
Outdated
Show resolved
Hide resolved
guides/common/modules/proc_exporting-the-library-environment-in-a-syncable-format.adoc
Outdated
Show resolved
Hide resolved
guides/common/modules/proc_exporting-the-library-environment-in-a-syncable-format.adoc
Outdated
Show resolved
Hide resolved
guides/common/modules/proc_exporting-the-library-environment-in-a-syncable-format.adoc
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* Remove steps from generating content It was decided that certain steps in generating content are not necessary due to conflicting statements about importability of syncable exports. * Fix snip file and add content Snip file restored and info from snip file that needed to be removed specifically for this section to avoid contradictory statements was put in the section. * Content cleanup for clarity
* Remove steps from generating content It was decided that certain steps in generating content are not necessary due to conflicting statements about importability of syncable exports. * Fix snip file and add content Snip file restored and info from snip file that needed to be removed specifically for this section to avoid contradictory statements was put in the section. * Content cleanup for clarity
* Remove steps from generating content It was decided that certain steps in generating content are not necessary due to conflicting statements about importability of syncable exports. * Fix snip file and add content Snip file restored and info from snip file that needed to be removed specifically for this section to avoid contradictory statements was put in the section. * Content cleanup for clarity
* Remove steps from generating content It was decided that certain steps in generating content are not necessary due to conflicting statements about importability of syncable exports. * Fix snip file and add content Snip file restored and info from snip file that needed to be removed specifically for this section to avoid contradictory statements was put in the section. * Content cleanup for clarity
* Remove steps from generating content It was decided that certain steps in generating content are not necessary due to conflicting statements about importability of syncable exports. * Fix snip file and add content Snip file restored and info from snip file that needed to be removed specifically for this section to avoid contradictory statements was put in the section. * Content cleanup for clarity
* Remove steps from generating content It was decided that certain steps in generating content are not necessary due to conflicting statements about importability of syncable exports. * Fix snip file and add content Snip file restored and info from snip file that needed to be removed specifically for this section to avoid contradictory statements was put in the section. * Content cleanup for clarity
* Remove steps from generating content It was decided that certain steps in generating content are not necessary due to conflicting statements about importability of syncable exports. * Fix snip file and add content Snip file restored and info from snip file that needed to be removed specifically for this section to avoid contradictory statements was put in the section. * Content cleanup for clarity
* Remove steps from generating content It was decided that certain steps in generating content are not necessary due to conflicting statements about importability of syncable exports. * Fix snip file and add content Snip file restored and info from snip file that needed to be removed specifically for this section to avoid contradictory statements was put in the section. * Content cleanup for clarity
* Remove steps from generating content It was decided that certain steps in generating content are not necessary due to conflicting statements about importability of syncable exports. * Fix snip file and add content Snip file restored and info from snip file that needed to be removed specifically for this section to avoid contradictory statements was put in the section. * Content cleanup for clarity
It was decided that certain steps in generating content are not
necessary due to conflicting statements about
importability of syncable
exports.
Please cherry-pick my commits into: