Skip to content

Added support for CFScript - #2771

Merged
RunDevelopment merged 11 commits into
PrismJS:masterfrom
mjclemente:cfscript
Feb 19, 2021
Merged

RunDevelopment merged 11 commits into
PrismJS:masterfrom
mjclemente:cfscript

Conversation

@mjclemente

Copy link
Copy Markdown
Contributor

This partially deals with #2035, though does not resolve it entirely.

CFML has independent tag and script based syntaxes. This adds support for script syntax.

Here are some language references:

@github-actions

github-actions Bot commented Feb 18, 2021 •

Copy link
Copy Markdown

JS File Size Changes (gzipped)

A total of 3 files have changed, with a combined diff of +720 B (+16.3%).

Details
file master pull size diff % diff
components/prism-cfscript.min.js 0 Bytes 699 B +699 B +100.0%
plugins/autoloader/prism-autoloader.min.js 2.19 KB 2.19 KB +10 B +0.5%
plugins/show-language/prism-show-language.min.js 2.23 KB 2.24 KB +11 B +0.5%

Generated by 🚫 dangerJS against 7695c65

@RunDevelopment RunDevelopment left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good work @mjclemente!

A couple of minor things and then this is ready.

Comment thread components/prism-cfscript.js Outdated
Comment thread components/prism-cfscript.js
Comment thread components/prism-cfscript.js Outdated
Comment thread components/prism-cfscript.js Outdated
Comment thread components/prism-cfscript.js Outdated
Comment thread components/prism-cfscript.js Outdated
Comment thread components/prism-cfscript.js
@mjclemente

Copy link
Copy Markdown
Contributor Author

Thanks for the fast, thorough, and really helpful review 😀

I believe I've made all requested changes.

@RunDevelopment RunDevelopment left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you for the quick response!

One more comment. Also, could you please add the link to https://cfdocs.org/script in a comment at the beginning of the file? This will be helpful to future contributors.

Comment thread components.json Outdated
@mjclemente

Copy link
Copy Markdown
Contributor Author

Makes sense!

Changes made. Again, thanks for the prompt and thorough handling!

@RunDevelopment
RunDevelopment merged commit b0a6ec8 into PrismJS:master Feb 19, 2021
@RunDevelopment

Copy link
Copy Markdown
Member

Thank you for contributing @mjclemente!

@mjclemente

Copy link
Copy Markdown
Contributor Author

Thank you! 🎉

@dawesi

dawesi commented Sep 7, 2025 •

Copy link
Copy Markdown

can we also create a branch of cfml for cfml/lucee (which boxlang also supports that syntax)

This is divergent from Adobe CFML/CFScript now also, as it containers many functions / helpers methods that are not in coldfusion and vice versa.

They publish the latest docs in json files zipped here.

https://docs.lucee.org/lucee-docs-json.zip

@mjclemente

Copy link
Copy Markdown
Contributor Author

Given that Lucee/Boxlang/Adobe ColdFusion are all CFML, my first thought would be that they could all be incorporated here. While there may be some different tags, I don't see any issues in including them all - they have more in common than different (at least Lucee CFML and Adobe). Not sure what functions are missing at this point.

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.

3 participants