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-1348)-Conversion of erb to epp templates except mod or vhost dir… #2448

Closed

Conversation

praj1001
Copy link
Contributor

@praj1001 praj1001 commented Sep 4, 2023

Summary

Conversion of ERB templates to EPP for apache module. It doesn't converts any erb template present in mod or vhost directory.

Additional Context

Related Issues (if any)

Checklist

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

@praj1001 praj1001 added the bugfix label Sep 4, 2023
@praj1001 praj1001 force-pushed the CAT-1348_erb_to_epp_except_in_mod_vhost_directory branch 2 times, most recently from d8e5e67 to e605fce Compare September 4, 2023 09:33
@ekohl
Copy link
Collaborator

ekohl commented Sep 4, 2023

FYI: I did start https://github.com/ekohl/erb2epp and ekohl/erb2epp#4 does improve things a bit, but I haven't finished that review. Perhaps it can help you.

templates/fastcgi/server.epp Outdated Show resolved Hide resolved
@praj1001 praj1001 force-pushed the CAT-1348_erb_to_epp_except_in_mod_vhost_directory branch 5 times, most recently from 8e8bd66 to 88e941c Compare September 5, 2023 07:39
@dinesh-gujar dinesh-gujar force-pushed the CAT-1348_erb_to_epp_except_in_mod_vhost_directory branch 2 times, most recently from 9775b39 to a9b84c3 Compare September 5, 2023 07:51
@dinesh-gujar dinesh-gujar force-pushed the CAT-1348_erb_to_epp_except_in_mod_vhost_directory branch from 5abf162 to be506d4 Compare September 5, 2023 07:54
@praj1001 praj1001 closed this Sep 5, 2023
@praj1001 praj1001 deleted the CAT-1348_erb_to_epp_except_in_mod_vhost_directory branch September 5, 2023 10:45
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