Skip to content

Conversation

@rymdbar
Copy link
Contributor

@rymdbar rymdbar commented Nov 22, 2025

Remove bogus test of non-existing g:ale_html_superhtml_options.

As suggested and ignored in comment to #5055.

Remove bogus test of non-existing g:ale_html_superhtml_options.
@rymdbar rymdbar requested a review from hsanson November 22, 2025 13:44
@rymdbar
Copy link
Contributor Author

rymdbar commented Nov 22, 2025

It seems it is not possible to add @yoan667 as a reviewer here, but a review from that account would be practically required.

@yoan667
Copy link
Contributor

yoan667 commented Nov 22, 2025

Hello @rymdbar, sorry for the oversight. I had no intention of ignoring your feedback. I thought I had resolved the issue in the Vader test file, but I must have forgotten to push the change. I’ll be more careful next time, and again, I'm sorry.

@rymdbar
Copy link
Contributor Author

rymdbar commented Nov 22, 2025

Hello @rymdbar, sorry for the oversight.

It might actually be me who ought to apologize for having phrased myself unnecessarily harsh. Sorry about that, @yoan667!

I do get angry when these things happen, but that anger should be directed at Microsoft who provides this horrible platform. Wouldn't be great if ALE was hosted on some forge which helps with reviews rather than does it best of hiding and hindering clear communication‽ Even Atlassian gets that working better in Bitbucket for Christs sake, and that's not exactly an example of anything done right…

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants