discussion: Lint Jelly files?

[janfaracik]

For discussion purposes -

How do we feel about linting Jelly? Currently nigh every other file is linted in Jenkins and this works well. I'd propose including Jelly in that too, I've tweaked .prettierrc to include Jelly files and the output seems to be quite sensible.

@jenkinsci/sig-ux

[NotMyFault]

No objections from me, if linting works consistently 👍

[timja]

Fine with me if you can get a green build

[github-actions[bot]]

Please take a moment and address the merge conflicts of your pull request. Thanks!

[timja]

The linting is breaking something

[github-actions[bot]]

Please take a moment and address the merge conflicts of your pull request. Thanks!

[janfaracik]

So the tests are failing as certain xpath selectors are no longer valid due to the whitespace changes, e.g. Add parameter is now Add parameter.

This can be fixed by setting htmlWhitespaceSensitivity to strict, see https://prettier.io/docs/en/options.html#html-whitespace-sensitivity and https://prettier.io/blog/2018/11/07/1.15.0#whitespace-sensitive-formatting. This brings its own issues however as it seems to be mangling rss20.jelly and it's also no longer as opinionated.

[timja]

So the tests are failing as certain xpath selectors are no longer valid due to the whitespace changes, e.g. Add parameter is now Add parameter.

This can be fixed by setting htmlWhitespaceSensitivity to strict, see prettier.io/docs/en/options.html#html-whitespace-sensitivity and prettier.io/blog/2018/11/07/1.15.0#whitespace-sensitive-formatting. This brings its own issues however as it seems to be mangling rss20.jelly and it's also no longer as opinionated.

This normally fixes xpath that is that sensitive: https://developer.mozilla.org/en-US/docs/Web/XPath/Functions/normalize-space

[github-actions[bot]]

Please take a moment and address the merge conflicts of your pull request. Thanks!

[github-actions[bot]]

Please take a moment and address the merge conflicts of your pull request. Thanks!

[github-actions[bot]]

Please take a moment and address the merge conflicts of your pull request. Thanks!

[github-actions[bot]]

Please take a moment and address the merge conflicts of your pull request. Thanks!

[github-actions[bot]]

Please take a moment and address the merge conflicts of your pull request. Thanks!

[github-actions[bot]]

Please take a moment and address the merge conflicts of your pull request. Thanks!

[janfaracik]

Going to close this - there's too many oddities after linting sadly. Will investigate again in the future if there's a better way. Thanks to those that commented.