Properly specify lang tags

[PerchunPak]

#261 attempted to introduce language tags, but that did not work for me (in NeoVim, it did not highlight code blocks as expected) and raised a bunch of deprecation warnings by Lix. This introduces them properly (and btw please consider using nixfmt, it is already de-jure standart and adopted in nixpkgs)

warning: Whitespace calculations for indentation stripping in a multiline ''-string include the first line, so putting text on it will effectively disable all indentation stripping. To fix this, simply break the line right after the string starts. Use --extra-deprecated-features broken-string-indentation to silence this warning.
         at /nix/store/wxlsi86bzplwnfqvcaxmcc686prncn4n-source/hm-module/places.nix:502:85:
            501|
            502|           updateScript = pkgs.writeShellScript "zen-sessions-update-${profileName}" ''            # bash
               |                                                                                     ^
            503|             SESSIONS_FILE="${sessionsFile}"

[luisnquin]

Now it doesn't highlight the code blocks on my end. Lix shouldn't care about it.

[luisnquin]

Two options here:

1. Remove the language tags to avoid the Lix warnings.

2. Introduce nixfmt in this repository, although language tags will not work for me until I switch my editor extension.

[PerchunPak]

Now it doesn't highlight the code blocks on my end. Lix shouldn't care about it.

What are you using?

[luisnquin]

Now it doesn't highlight the code blocks on my end. Lix shouldn't care about it.

What are you using?

This extension

[PerchunPak]

Can you try https://github.com/nix-community/vscode-nix-ide? This is the LSP for Nix

[luisnquin]

Can you try nix-community/vscode-nix-ide? This is the LSP for Nix

I'm already using it

[PerchunPak]

You are using nil, which is IMO worse and has less features. Change nix.serverPath to nixd

[aw1cks]

Hi, any further thoughts?
I would love to see it merged, I recently switched to Lix and the warnings are quite verbose

warning: Whitespace calculations for indentation stripping in a multiline ''-string include the first line, so putting text on it will effectively disable all indentation stripping. To fix this, simply break the line right after the string starts. Use --extra-deprecated-features broken-string-indentation to silence this warning.
         at /nix/store/asprs7j92yx6z2gjm7vv6m05lxhykvps-source/hm-module/places.nix:502:85:
            501|
            502|           updateScript = pkgs.writeShellScript "zen-sessions-update-${profileName}" ''            # bash
               |                                                                                     ^
            503|             SESSIONS_FILE="${sessionsFile}"
warning: Whitespace calculations for indentation stripping in a multiline ''-string include the first line, so putting text on it will effectively disable all indentation stripping. To fix this, simply break the line right after the string starts. Use --extra-deprecated-features broken-string-indentation to silence this warning.
         at /nix/store/asprs7j92yx6z2gjm7vv6m05lxhykvps-source/hm-module/places.nix:571:96:
            570|         in
            571|           nameValuePair "zen-sessions-${profileName}" (lib.hm.dag.entryAfter ["writeBoundary"] ''            # bash
               |                                                                                                ^
            572|             ${updateScript}
warning: Whitespace calculations for indentation stripping in a multiline ''-string include the first line, so putting text on it will effectively disable all indentation stripping. To fix this, simply break the line right after the string starts. Use --extra-deprecated-features broken-string-indentation to silence this warning.
         at /nix/store/asprs7j92yx6z2gjm7vv6m05lxhykvps-source/hm-module/mods.nix:63:85:
             62|
             63|           updateModsScript = pkgs.writeShellScript "zen-mods-update-${profileName}" ''            # bash
               |                                                                                     ^
             64|                       THEMES_FILE="${themesFilePath}"
warning: Whitespace calculations for indentation stripping in a multiline ''-string include the first line, so putting text on it will effectively disable all indentation stripping. To fix this, simply break the line right after the string starts. Use --extra-deprecated-features broken-string-indentation to silence this warning.
         at /nix/store/asprs7j92yx6z2gjm7vv6m05lxhykvps-source/hm-module/sine.nix:107:94:
            106|
            107|           updateSineModsScript = pkgs.writeShellScript "zen-sine-mods-update-${profileName}" ''            # bash
               |                                                                                              ^
            108|             MODS_FILE="${modsFilePath}"
warning: Whitespace calculations for indentation stripping in a multiline ''-string include the first line, so putting text on it will effectively disable all indentation stripping. To fix this, simply break the line right after the string starts. Use --extra-deprecated-features broken-string-indentation to silence this warning.
         at /nix/store/asprs7j92yx6z2gjm7vv6m05lxhykvps-source/hm-module/sine.nix:274:97:
            273|         in
            274|           nameValuePair "zen-sine-mods-${profileName}" (lib.hm.dag.entryAfter ["writeBoundary"] ''            # bash
               |                                                                                                 ^
            275|             ${updateSineModsScript}
warning: Whitespace calculations for indentation stripping in a multiline ''-string include the first line, so putting text on it will effectively disable all indentation stripping. To fix this, simply break the line right after the string starts. Use --extra-deprecated-features broken-string-indentation to silence this warning.
         at /nix/store/asprs7j92yx6z2gjm7vv6m05lxhykvps-source/package.nix:64:19:
             63|
             64|   installDarwin = ''    # bash
               |                   ^
             65|     runHook preInstall
warning: Whitespace calculations for indentation stripping in a multiline ''-string include the first line, so putting text on it will effectively disable all indentation stripping. To fix this, simply break the line right after the string starts. Use --extra-deprecated-features broken-string-indentation to silence this warning.
         at /nix/store/asprs7j92yx6z2gjm7vv6m05lxhykvps-source/package.nix:100:18:
             99|
            100|   installLinux = ''    # bash
               |                  ^
            101|     runHook preInstall

[luisnquin]

Okay, why not? I won't say no to these kind of changes anymore :P

[aw1cks]

awesome, thanks!