-
-
Notifications
You must be signed in to change notification settings - Fork 38
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
Add Dockerfile highlighting support #135
Conversation
{ | ||
public function getName(): string | ||
{ | ||
return 'Dockerfile'; |
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.
Should this be dockerfile
? Maybe add docker
as an alias as well?
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.
I had the same thought as you regarding the alias. And ChatGPT would also describe it this way, so I think dockerfile
and docker
will be enough.
Let me know your thoughts about that one comment, otherwise I'm good merging it 👍 |
Pull Request Test Coverage Report for Build 10227678571Details
💛 - Coveralls |
46721e7
to
cd9156b
Compare
Cool, thanks! |
Based on keywords listed here: https://docs.docker.com/reference/dockerfile/#overview
With support for
AS
keyword when aliasing image names, and highlighting image names and tags: