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

Mark operator like functions as functions in scheme #1467

Merged
merged 4 commits into from
Mar 3, 2019

Conversation

jcubic
Copy link
Contributor

@jcubic jcubic commented Jul 8, 2018

This one more issue fix found in #1451

this one mark (** 10) as function not operator * and then blank * 

there is (?=\s)|$ in operator so it match for space look ahead but also end of the string so it match operator test file. I've also added the same to functions so it's highlighted when you start typing function call like (fun this will be mark as function.

Copy link
Member

@RunDevelopment RunDevelopment left a comment

Choose a reason for hiding this comment

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

Sorry for the delay @jcubic!

The changes look good, just a small nit.

components/prism-scheme.js Outdated Show resolved Hide resolved
components/prism-scheme.js Outdated Show resolved Hide resolved
@jcubic
Copy link
Contributor Author

jcubic commented Mar 3, 2019

Updated the code as requested, tests are passing (it seems that travis is enabled only on gh-pages).

@RunDevelopment
Copy link
Member

Thank you!
But this still needs rebasing before we can merge this.

I don't know why Travis is taking the day off but it should test your branch.

@RunDevelopment
Copy link
Member

Perfect!

Thank you for contributing!

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