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

Rename Dockerfile.tmpl to template.Dockerfile #1817

Merged

Conversation

f1rstmehul
Copy link
Contributor

What does this change

Change the generated dockerfile template to use .Dockerfile file extension to get syntax highlighting in IDEs.

What issue does it fix

Closes #1771

Notes for the reviewer

N/A

Checklist

  • Unit Tests
  • Documentation
  • Schema (porter.yaml)

If this is your first pull request, please add your name to the bottom of our Contributors list. Thank you for making Porter better! 🙇‍♀️

Copy link
Member

@carolynvs carolynvs left a comment

Choose a reason for hiding this comment

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

Looking good! 👍 I tried it out locally and it worked fine, other than the my comment below.

use `.Dockerfile` file extension to get syntax
highlighting in IDEs.

Signed-off-by: Parmeshwar Patidar <prm.mehul@gmail.com>
@f1rstmehul f1rstmehul force-pushed the change-dockerfile-extension branch from e2b921e to 7b98f50 Compare November 10, 2021 16:32
Copy link
Member

@carolynvs carolynvs left a comment

Choose a reason for hiding this comment

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

Thank you for knocking another item off of our v1 milestone! 🚂

@carolynvs carolynvs merged commit 3fd1dba into getporter:release/v1 Nov 10, 2021
@f1rstmehul f1rstmehul deleted the change-dockerfile-extension branch November 11, 2021 05:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants