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

enh(stan) updated with improved coverage of language keywords and patterns. #1829

Merged
merged 10 commits into from Dec 24, 2019

Conversation

jrnold
Copy link
Contributor

@jrnold jrnold commented Sep 9, 2018

  • Almost complete rewrite. Corrected most patterns and updated function
    definitions.
  • Add "stanfuncs" as an alias

This PR fixes the botched rebase in #1755.

-  Almost complete rewrite. Corrected most patterns and updated function 
definitions.
-  Add "stanfuncs" as an alias
@marcoscaceres
Copy link
Contributor

Thanks @jrnold! Would you by chance be willing to be the maintainer of Stan's syntax highlighting? If so, I can spin up a repository in the highlightjs organization. It would mean you could update the language independently of hljs core.

@joshgoebel
Copy link
Member

@jrnold You still out there? Any thoughts about being maintainer for this?

Copy link
Member

@joshgoebel joshgoebel left a comment

Choose a reason for hiding this comment

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

Can you also add a note in CHANGES.md?

src/languages/stan.js Show resolved Hide resolved
@joshgoebel joshgoebel added this to the 9.15.11 milestone Oct 11, 2019
@egor-rogov
Copy link
Collaborator

This language is already in the core. Why should we move it away?

@joshgoebel
Copy link
Member

Moving this to the language discussion.

@joshgoebel
Copy link
Member

Since this is already in core though I was moving forward with reviewing it as if it was mergeable since it looks good to me... what to do long-term is a separate discussion.

@joshgoebel joshgoebel modified the milestones: 9.15.11, 9.16 Oct 24, 2019
@joshgoebel
Copy link
Member

Bumping this from 9.15.11.

@joshgoebel joshgoebel added this to Investigate or close in Highlight.js Oct 25, 2019
@joshgoebel joshgoebel removed this from the 9.16 milestone Oct 29, 2019
@joshgoebel joshgoebel moved this from Promote or close to Backlog in Highlight.js Oct 29, 2019
@joshgoebel
Copy link
Member

@jrnold You still around? Would love to work on this with you and get it merged...

@joshgoebel
Copy link
Member

@jrnold Ping.

src/languages/stan.js Show resolved Hide resolved
src/languages/stan.js Show resolved Hide resolved
@joshgoebel joshgoebel changed the title Update Stan language highlighter enh(stan) updated with improved coverage of language keywords and patterns. Dec 24, 2019
@joshgoebel joshgoebel merged commit 85db1cf into highlightjs:master Dec 24, 2019
Highlight.js automation moved this from Backlog to Done Dec 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Highlight.js
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants