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

Wrong colorization in Yaml files #3

Open
sugdyzhekov opened this issue Jul 7, 2016 · 6 comments
Open

Wrong colorization in Yaml files #3

sugdyzhekov opened this issue Jul 7, 2016 · 6 comments
Assignees
Milestone

Comments

@sugdyzhekov
Copy link

Could you check following examples? Looks like Jinja colorization works wrong.

Example 1:
example1

setup_plesk:
  cmd.run:
    - name: |
        PSA_PASSWORD={{ pass }} /usr/sbin/plesk bin init_conf --init -shared_ips add:{{ ip }} -license_agreed true -admin_info_not_required true -send_announce false -send_tech_announce false -hostname {{ fqdn }} -passwd "" -company plesk.ru -name Administrator -email {{ email }}
        ...

Example 2:
example2

setup_plesk:
  cmd.run:
    - name: PSA_PASSWORD={{ pass }} /usr/sbin/plesk bin init_conf --init -shared_ips add:{{ ip }} -license_agreed true -admin_info_not_required true -send_announce false -send_tech_announce false -hostname {{ fqdn }} -passwd "" -company plesk.ru -name Administrator -email {{ email }}

Example 3:
example3

create_default_subscription:
  cmd.run:
    - name: /usr/sbin/plesk bin subscription -c {{ fqdn }} -hosting true -ip {{ ip }} -login support -passwd $(/usr/bin/openssl rand -base64 12)
    - unless: /usr/sbin/plesk bin subscription -i {{ fqdn }} &>/dev/null
@wholroyd wholroyd self-assigned this Jul 8, 2016
@wholroyd
Copy link
Owner

wholroyd commented Jul 8, 2016

Thanks for letting me know about the issue.

The way VS Code handles languages is different depending on if they are core to the editor or come as an extension. Extensions can't override core languages very easily and have to use hackery sometimes to get things to work.

Give me a couple of days to take a look at it to see how to implement the changes.

@wholroyd
Copy link
Owner

To expand a bit on my last comment, see microsoft/vscode#1751 for more information on what I mean.

I can't immediately fix this issue without rewriting the YAML syntax from scratch or until Microsoft fixes the language override issue. This will take some time to fix, but I can do it.

@wholroyd wholroyd added this to the 1.0 milestone Jul 15, 2016
@wholroyd wholroyd added the bug label Jul 15, 2016
@wholroyd
Copy link
Owner

At this point, I don't see myself being able to rewrite the YAML language to make this work and this is technically not a bug with the plugin, but a limitation of Code. As such, there is nothing I can really do about this until I get time or they fix language overrides in Code. I'm closing as Won't Fix for now.

@sugdyzhekov
Copy link
Author

Ok, I got it. Thank you for your efforts!

чт, 10 нояб. 2016 г. в 9:51, William Holroyd notifications@github.com:

At this point, I don't see myself being able to rewrite the YAML language
to make this work and this is technically not a bug with the plugin, but a
limitation of Code. As such, there is nothing I can really do about this
until I get time or they fix language overrides in Code. I'm closing as
Won't Fix for now.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#3 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIilWZ8AtiT7ISHjP_q1IPZrGpdXE2uTks5q8obJgaJpZM4JGvMS
.

@wholroyd
Copy link
Owner

Wanted to let you know that VSCode added support for embedded and injected languages recently. How does this look?

image

@wholroyd wholroyd removed the wontfix label Nov 21, 2016
@wholroyd wholroyd added this to the 1.0 milestone Nov 21, 2016
@wholroyd wholroyd reopened this Nov 21, 2016
@wholroyd
Copy link
Owner

wholroyd commented Oct 2, 2017

Ended up getting blocked on this as it didn't work in YAML for all scenarios, so it's on pause.

Linking to #19.

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

No branches or pull requests

2 participants