a couple of issues

[ctf0]

• the "htmlhint.documentSelector wont work because of

  vscode-htmlhint/htmlhint/package.json

  Line 32 in 07db8b2

  "onLanguage:html"

• htmlhint.configFile isnt available

  vscode-htmlhint/htmlhint/package.json

  Line 39 in 07db8b2

  "properties": {

REF:
https://github.com/htmlhint/vscode-htmlhint/tree/main/htmlhint#settings

[coliff]

Thanks for reporting these issues!

• On that first issue, is the solution to change the activationEvents to onFileSystem:file like you did on your fork? Does that work as expected with no problems?
• For the second do you have a suggested fix for htmlhint.configFile?

[ctf0]

both of the issues are solved on my branch yes, as for does it work https://marketplace.visualstudio.com/items?itemName=ctf0.htmlhint the installation count says it all especially that this is all what i've added

[michael-nok]

@coliff Could you please include just the activationEvents changes provided by @ctf0 to allow for more startup options?

[rubensa]

Any work on this? I'm trying to validate some html inside markdown documents.

I think that the:

    "activationEvents": [
        "onFileSystem:file"
    ],

is needed so the extension can be activated for other files than HTML (so it can be used for markdown files if I specify markdown inside htmlhint.documentSelector configuration) .

The other change I can see is the addition of:

                "htmlhint.documentSelector": {
                    "items": {
                        "type": "string"
                    }
                }

that I don't know if it is really necessary.

[rubensa]

@coliff For the htmlhint.configFile part @ctf0 added the htmlhint.optionsFile configuration.

In htmlhint-server/src/server.ts he added optionsFile to the Settings interface:

interface Settings {
    htmlhint: {
        enable: boolean;
        options: any;
        optionsFile: string;
    },
    [key: string]: any;
}

and the code to load the configuration if the option is specified inside getConfiguration:

    else if (settings.htmlhint && settings.htmlhint.optionsFile) {
        options = loadConfigurationFile(settings.htmlhint.optionsFile)
    }

[coliff]

sorry for the delay on this... I'd been testing that change locally and hope to get an update with this change out soon. thanks!