Skip to content

Add record and var keywords to Groovy syntax#3872

Merged
joshgoebel merged 4 commits intohighlightjs:mainfrom
glaforge:patch-1
Oct 8, 2023
Merged

Add record and var keywords to Groovy syntax#3872
joshgoebel merged 4 commits intohighlightjs:mainfrom
glaforge:patch-1

Conversation

@glaforge
Copy link
Copy Markdown
Contributor

Groovy added support for records like Java, and also allows the use of var (in addition to def) to define variables or specified non-typed arguments.

@github-actions
Copy link
Copy Markdown

Build Size Report

Changes to minified artifacts in /build, after gzip compression.

5 files changed

Total change +10 B

View Changes
file base pr diff
es/core.min.js 8.13 KB 8.13 KB -1 B
es/highlight.min.js 8.13 KB 8.13 KB -1 B
es/languages/groovy.min.js 949 B 954 B +5 B
highlight.min.js 8.16 KB 8.16 KB -1 B
languages/groovy.min.js 951 B 959 B +8 B

@joshgoebel
Copy link
Copy Markdown
Member

Looks good, but please update CHANGES.md as well.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 8, 2023

Build Size Report

Changes to minified artifacts in /build, after gzip compression.

2 files changed

Total change +13 B

View Changes
file base pr diff
es/languages/groovy.min.js 949 B 954 B +5 B
languages/groovy.min.js 951 B 959 B +8 B

@glaforge
Copy link
Copy Markdown
Contributor Author

glaforge commented Oct 8, 2023

Sorry joshgoebel for missing the CHANGES.md update, now we should be good!

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 8, 2023

Build Size Report

Changes to minified artifacts in /build, after gzip compression.

5 files changed

Total change +16 B

View Changes
file base pr diff
es/core.min.js 8.13 KB 8.13 KB +1 B
es/highlight.min.js 8.13 KB 8.13 KB +1 B
es/languages/groovy.min.js 949 B 954 B +5 B
highlight.min.js 8.16 KB 8.16 KB +1 B
languages/groovy.min.js 951 B 959 B +8 B

CHANGES.md Outdated
- enh(php) detect newer more flexible NOWdoc syntax (#3679) [Timur Kamaev][]
- enh(python) improve autodetection of code with type hinting any function's return type (making the `->` operator legal) [Keyacom][]
- enh(bash) add `select` and `until` as keywords
- enh(groovy) add `record` and `var` as keywords [Guillaume Laforge][]
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.

You need to also add a subscript link on lines ~ 102-120.

@glaforge
Copy link
Copy Markdown
Contributor Author

glaforge commented Oct 8, 2023

I moved in the unreleased version as well, I realised I put it in the already released version

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 8, 2023

Build Size Report

Changes to minified artifacts in /build, after gzip compression.

5 files changed

Total change +10 B

View Changes
file base pr diff
es/core.min.js 8.13 KB 8.13 KB -1 B
es/highlight.min.js 8.13 KB 8.13 KB -1 B
es/languages/groovy.min.js 949 B 954 B +5 B
highlight.min.js 8.16 KB 8.16 KB -1 B
languages/groovy.min.js 951 B 959 B +8 B

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 8, 2023

Build Size Report

Changes to minified artifacts in /build, after gzip compression.

5 files changed

Total change +7 B

View Changes
file base pr diff
es/core.min.js 8.13 KB 8.13 KB -2 B
es/highlight.min.js 8.13 KB 8.13 KB -2 B
es/languages/groovy.min.js 949 B 954 B +5 B
highlight.min.js 8.16 KB 8.16 KB -2 B
languages/groovy.min.js 951 B 959 B +8 B

@joshgoebel joshgoebel merged commit fa9a62c into highlightjs:main Oct 8, 2023
@joshgoebel
Copy link
Copy Markdown
Member

Thanks!

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