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

TOML support #1488

Merged
merged 7 commits into from
Dec 2, 2018
Merged

TOML support #1488

merged 7 commits into from
Dec 2, 2018

Conversation

RunDevelopment
Copy link
Member

This PR adds support for Tom's Obvious, Minimal Language.

I made some simplifications:

  1. The .s of dotted keys are not highlighted. This is because the . could be inside quotes and checking that isn't easy.
  2. There is no distinction between table [foo] and array [[foo]]. foo will be highlighted as table in both cases.

This PR resolves #1436

Copy link
Member

@mAAdhaTTah mAAdhaTTah left a comment

Choose a reason for hiding this comment

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

I'm going to leave reviewing the regex here to my co-maintainers, but there's some tests missing here as well (table & comment).

@RunDevelopment
Copy link
Member Author

@mAAdhaTTah

Test were added.

I'm looking forward to the review.

@RunDevelopment
Copy link
Member Author

RunDevelopment commented Jul 21, 2018

@mAAdhaTTah

Isn't the label supposed to be needs review ?

@mAAdhaTTah
Copy link
Member

Yeah, switched to needs review.

@gqcn
Copy link

gqcn commented Sep 5, 2018

How it is going, I am quite looking forward to this feature.

@RunDevelopment
Copy link
Member Author

@johng-cn
Well, I'm looking forward to this too but someone has to thoroughly review it. Unfortunately, @Golmote is quite busy right now, so it might take a little while.

@mAAdhaTTah
Copy link
Member

If we get the conflicts resolved, we can merge this.

@mAAdhaTTah mAAdhaTTah merged commit 5b6ad70 into PrismJS:master Dec 2, 2018
@verythorough
Copy link

Thanks, @RunDevelopment!

ggrossetie pushed a commit to ggrossetie/prism that referenced this pull request Mar 11, 2019
Add support for [Tom's Obvious, Minimal Language](https://github.com/toml-lang/toml).

Some simplifications:

1. The `.`s of dotted keys are not highlighted. This is because the `.` could be inside
quotes and checking that isn't easy.
2. There is no distinction between table `[foo]` and array `[[foo]]`. `foo` will be
highlighted as `table` in both cases.
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.

Support for TOML
4 participants