search

cylc / cylc-textmate-grammar

TextMate grammar for Cylc workflow configuration files
BSD 3-Clause "New" or "Revised" License
0 stars 5 forks source link

Remove variable-length lookbehinds #49

Closed MetRonnie closed 4 months ago

MetRonnie commented 4 months ago

This is needed to allow the Textmate grammar to be added to GitHub Linguist. Currently:

Invalid regex in grammar: source.cylc (in Syntaxes/cylc.tmLanguage) contains a malformed regex (regex "(?<=(\b\w[\w\+\-@%]*|((:)([\w\-]...": lookbehind assertion is not fixed length (at offset 43))

https://github.com/oliver-sanders/linguist/actions/runs/8569942718/job/23486858235#step:4:78

This unfortunately makes the grammar slightly less able to handle Jinja2 cropping up anywhere, but worth the tradeoff

oliver-sanders commented 4 months ago

I've given in trying to get the linguist compiler to run for this repo in CI, let's try merging this and see what it spits out next time.