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

(CAT-1143)-Conversion_of_erb_templates_to_epp #944

Merged
merged 1 commit into from
Aug 10, 2023

Conversation

praj1001
Copy link
Contributor

@praj1001 praj1001 commented Aug 7, 2023

Summary

Conversion of ERB To EPP for templates in docker module.

Additional Context

Related Issues (if any)

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.
  • Manually verified. (For example puppet apply)

@praj1001 praj1001 added the bugfix label Aug 7, 2023
@praj1001 praj1001 requested a review from a team as a code owner August 7, 2023 09:11
Copy link
Contributor

@Ramesh7 Ramesh7 left a comment

Choose a reason for hiding this comment

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

LGTM

can we get one more review before we merge, as its big PR?

Copy link
Contributor

@LukasAud LukasAud left a comment

Choose a reason for hiding this comment

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

LGTM

@Ramesh7 Ramesh7 merged commit a61afc2 into main Aug 10, 2023
23 checks passed
@Ramesh7 Ramesh7 deleted the CAT-1143-erb_to_epp_docker branch August 10, 2023 10:28
smortex added a commit that referenced this pull request Jan 8, 2024
In #944, we switched from ERB to EPP.  This has not been released yet,
but while testing I spotted some white space changes that where
unexpected.

Adjust the code so that the EPP template produce the same result as the
ERB template did, making changes more relevant to end-users if they see
some files being changed.
Ramesh7 pushed a commit that referenced this pull request Jun 6, 2024
In #944, we switched from ERB to EPP.  This has not been released yet,
but while testing I spotted some white space changes that where
unexpected.

Adjust the code so that the EPP template produce the same result as the
ERB template did, making changes more relevant to end-users if they see
some files being changed.
kenyon added a commit to kenyon/puppetlabs-docker that referenced this pull request Jul 3, 2024
When these templates were converted from ERB to EPP in commit
3e101cb (puppetlabs#944), these lines were
swapped, causing a whitespace change and unnecessary service restarts.
kenyon added a commit to kenyon/puppetlabs-docker that referenced this pull request Jul 11, 2024
When these templates were converted from ERB to EPP in commit
3e101cb (puppetlabs#944), these lines were
swapped, causing a whitespace change and unnecessary service restarts.
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.

3 participants