Sometimes tidy does not lint html files on BufEnter event

[dense-analysis]

Sometimes tidy does not lint html files on BufEnter event

[w0rp]

Hmm, interesting. This will take a bit of debugging. I'll see if I can repeat the issue and fix it.

[w0rp]

Try again with the latest code. I added a very brief delay when the buffer is opened for checking the file, and that seems to fix the issue for me. I may need to adjust the delay in future.

[KabbAmine]

No, same thing.
IMO it doesn't look like a delay issue but a problem with either BufEnter autocmd or ALELint() function itself.

I'll investigate more.

[w0rp]

@KabbAmine Looks like GitHub automatically closed the issue based on my commit. I forgot that it does that. Check this out again. I realised that listening for BufEnter can trigger linting when a buffer is created, but before the file has been read into the buffer. So for some files, it was triggering the linting before the file was ready to be read. I added BufRead too to fix that. It might be worth switching to only BufRead. That may cause other issues. Have a look.

[KabbAmine]

Already tried with BufRead and BufReadPost in my local version (And your last commit) and still the same issue.

Calling ALELint works normally.

I'll try to debug that in the end of the day, and hopefully come with a fix or a workaround.

P.S: Don't commit to the master branch when we have such issues, create a dev-fixX branch and let work on it till the issue is solved.

[KabbAmine]

Ok, after some hours of head banging yesterday (literally 🤘) I've discovered that the issue is specific to the tidy installed on my system, so the bug is not reproducible somewhere else.
Using a bigger delay on BufEnter can help sometimes but still, the issue occurs randomly.

Let's wait a few days and if no one complains about tidy, we can consider the problem solved and close the issue.

[neersighted]

@w0rp I think it's safe to close this one.

[w0rp]

I'll close this issue. If the issue comes up again, we can increase the delay for running the linters on enter, as that seems to help.