@peterhashair opened this Pull Request on February 11th 2022 Contributor

Description:

remove nbsp

Review

@sgiehl commented on February 11th 2022 Member

Just out of curiosity, does that mean we shouldn't used   in Vue files at all? If so maybe we should document that somewhere

@diosmosis commented on February 11th 2022 Member

@sgiehl Seems like it, unless its ok to remove/not apply the specific test in ReleaseCheckListTest. Unfortunately I have no idea why its there.

I have it on my list to document it in developer-documentation. It's still possible to use it, but you'd have to use a computed property.

@sgiehl commented on February 11th 2022 Member

Guess it would otherwise be fine to ignore the compiled Vue files in that check. Iirc the check was there to prevent accidentally inserted invisible/special characters

@diosmosis commented on February 11th 2022 Member

I see, that's what I thought at first, but I guess I overthought.

This Pull Request was closed on February 11th 2022
Powered by GitHub Issue Mirror