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 the Cylc language. #6832

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

oliver-sanders
Copy link

Description

Add the Cylc language.

Checklist:

@oliver-sanders oliver-sanders requested a review from a team as a code owner May 16, 2024 11:57
@@ -0,0 +1,138 @@
#!Jinja2
Copy link
Author

Choose a reason for hiding this comment

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

I think the CI failure might be to do with this line?

 Cylc has a sample with an interpreter (Jinja2) that isn't explicitly defined in languages.yml

This shebang line is a functional part of the language (configures Cylc to run the Jinja2 preprocessor on this file before ingestion), but can remove it if necessary.

Choose a reason for hiding this comment

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

(The shebang doesn't affect tokenization of the file by the textmate grammar)

Copy link
Author

Choose a reason for hiding this comment

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

Removed the Jinja2 shebangs as they are causing issues with CI.

* These shebang lines are functional part of the Cylc language (they
  tell Cylc to run the Jinja2 preprocessor), however they cause issues
  with the tests as the Jinja2 preprocessor is not an interpreter which
  can be called on the CLI.
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.

None yet

3 participants