Skip to content

Conversation

@AmebaBrain
Copy link
Contributor

Added following commands to the bash language

  • docker
  • docker-compose
  • podman
  • podman-compose

According to guidelines minified file is not touched.

@github-actions
Copy link

github-actions bot commented Nov 28, 2021

JS File Size Changes (gzipped)

A total of 1 files have changed, with a combined diff of +19 B (+0.7%).

Details
file master pull size diff % diff
components/prism-bash.min.js 2.87 KB 2.89 KB +19 B +0.7%

Generated by 🚫 dangerJS against 57adfc3

@RunDevelopment
Copy link
Member

Thank you for the PR @AmebaBrain!

According to guidelines minified file is not touched.

When changing a language, the minified files are supposed to be updated. Seems like our docs need some clarification. Where did you read this?

@AmebaBrain
Copy link
Contributor Author

@RunDevelopment

2nd point in Contribute to Prism! section of readme says

Limit your changes to the unminified files in the components/ folder. prism.js and all minified files are also generated automatically by our build system.

Anyway, I've pushed the changes to the minified file as well. Because I did this from a new Windows based laptop please check whether line ending in a minified file is ok. If not, I will re-execute npm run build from Linux based machine day after tommorow.

@RunDevelopment
Copy link
Member

2nd point in Contribute to Prism! section of readme says

Good point. I'll clarify this tomorrow.

@RunDevelopment RunDevelopment merged commit 8c5ed25 into PrismJS:master Nov 29, 2021
@RunDevelopment
Copy link
Member

Thank you for contributing @AmebaBrain!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants