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

Add missing "object" highlighting to Pony definition #1932

Merged
merged 1 commit into from
Dec 26, 2018
Merged

Add missing "object" highlighting to Pony definition #1932

merged 1 commit into from
Dec 26, 2018

Conversation

SeanTAllen
Copy link
Contributor

The original Pony highlighting definition that Joe did is missing a Pony
keyword for highlighting like a class. After consulting with other Pony
committers and comparing results, we feel this small change is a big
improvement when highlighting Pony's object literals.

The original Pony highlighting definition that Joe did is missing a Pony
keyword for highlighting like a class. After consulting with other Pony
committers and comparing results, we feel this small change is a big
improvement when highlighting Pony's object literals.
Copy link
Collaborator

@egor-rogov egor-rogov left a comment

Choose a reason for hiding this comment

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

Makes sense.

@egor-rogov egor-rogov merged commit 5cd63a3 into highlightjs:master Dec 26, 2018
@egor-rogov
Copy link
Collaborator

@SeanTAllen Thanks, merged!
By the way, Pony lacks markup tests. PR is welcome (:

@SeanTAllen
Copy link
Contributor Author

Thanks @egor-rogov. I'm not familiar with Markup tests. Is there some documentation you can point me at? Any plans on another highlight release soon that this change would go out with?

@egor-rogov
Copy link
Collaborator

@SeanTAllen You can read this page (Testing Markup section): https://highlightjs.readthedocs.io/en/latest/building-testing.html
Actually we need more tests in `test/markup/pony/ to cover language syntax. This will made further improvements easier.

@egor-rogov
Copy link
Collaborator

As for the next release, we can ask @marcoscaceres about the plans.

@marcoscaceres
Copy link
Contributor

I'll try to push one out now.

@gregcline gregcline mentioned this pull request Jan 28, 2019
@marcoscaceres
Copy link
Contributor

Ok, 9.14.1 was released.

@SeanTAllen SeanTAllen deleted the pony-object branch January 31, 2019 19:56
@SeanTAllen
Copy link
Contributor Author

Thanks @marcoscaceres!

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.

None yet

3 participants