Skip to content

Expression syntax highlighting doesn't seem to respect comments #158

Open

Description

@max-sixty

Describe the bug

Here, this line should appear as a comment:

image

To Reproduce

Here's the text:

  update-rust-toolchain:
    runs-on: ubuntu-latest
    # Don't run on forks; will attempt to create a PR into the fork...

    # if: ${{ !github.event.pull_request.head.repo.fork }}

Extension Version
v0.25.6

Activity

  1. changed the title [-]`if` expression syntax highlighting doesn't seem to respect comments[/-] [+]Expression syntax highlighting doesn't seem to respect comments[/+] on May 8, 2023
  2. cschleiden commented on May 8, 2023

    @cschleiden
    Member

    Currently we always syntax highlight expressions in comments, that's not just limited to if expressions. It's due to the way we inject into the yaml grammar.

    Since adding GitHub Actions Workflows as a separate language we could modify the yaml tm grammar if we wanted to.

  3. muzimuzhi commented on May 9, 2023

    @muzimuzhi
    Contributor

    Since adding GitHub Actions Workflows as a separate workflow we could modify the yaml tm grammar if we wanted to.

    Did you mean "as a separate language"?

  4. moved this from Backlog 🗒 to Triaged 📝 in GitHub Actions VS Code Extensionon May 15, 2023
  5. scott-doyland-burrows commented on May 31, 2023

    @scott-doyland-burrows

    I noticed this as well, this just looks odd:

    image

    The entire comment should be shown in green, in my opinion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions