Skip to content

Commit 686bdcf

Browse files
authored
fix: remove stray f-string sub-rule that mis-highlights PRQL f-strings (#6095)
1 parent d86d0ef commit 686bdcf

2 files changed

Lines changed: 0 additions & 20 deletions

File tree

web/book/highlight-prql.js

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -14,14 +14,6 @@ Website: https://prql-lang.org/
1414

1515
// Inspired by [Pest's book](https://github.com/pest-parser/book)
1616

17-
// mdBook exposes a minified version of highlight.js, so the language
18-
// definition objects below have abbreviated property names:
19-
// "b" => begin
20-
// "e" => end
21-
// "c" => contains
22-
// "k" => keywords
23-
// "cN" => className
24-
2517
// TODO:
2618
// - Can we represent strings with the actual rule of >= 3 quotes?
2719
// - Aliases seem a bit strong?
@@ -209,12 +201,6 @@ const formatting = function (hljs) {
209201
],
210202
contains: [
211203
CHAR_ESCAPE,
212-
{
213-
scope: "variable",
214-
begin: "f",
215-
end: '"',
216-
// excludesEnd: true,
217-
},
218204
// TODO: would be nice to have this be a different color, but I don't
219205
// think it's possible to have a subscope within the begin / end.
220206
// {

web/website/themes/prql-theme/static/plugins/highlight/prql.js

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -197,12 +197,6 @@ const formatting = function (hljs) {
197197
],
198198
contains: [
199199
CHAR_ESCAPE,
200-
{
201-
scope: "variable",
202-
begin: "f",
203-
end: '"',
204-
// excludesEnd: true,
205-
},
206200
// TODO: would be nice to have this be a different color, but I don't
207201
// think it's possible to have a subscope within the begin / end.
208202
// {

0 commit comments

Comments
 (0)