fixed escaping characters for library.properties

[vincentsijben]

No description provided.

[mingness]

looks great!

[Stefterv]

This issue should be solved at the contributions level

[mingness]

@Stefterv I agree this should be corrected in the contributions repo, but this i still an issue that should be solved here - the library.properties file should not have escaped colons.

[Stefterv]

Hi @mingness we looked into it, and actually the escaped colon is correct according the .properties specification.

[mingness]

@Stefterv ok, does it break the process if it's not escaped? if not, maybe now's a good time to make it more intuitive?

[mingness]

@Stefterv actually, which specification are you referring to above?
The Processing specification is without the escape: https://github.com/processing/processing4/wiki/Library-Basics#DescribingYourLibrary

[SableRaf]

The .properties format is defined as part of Java, as described here, where escaping : is technically correct. This is because : can be used interchangeably with =.

Not escaping: doesn't break our pipeline for library.properties files as described in the old Library Basics page (which is probably fine for human-generated properties as it does make it more intuitive) but we should thrive to stick to standards as much as possible when automating things.

[mingness]

While there is a Processing standard, I would default to that. I'm not clear what further action is necessary other than merging or not merging. I will leave the decision then to you two.

[mingness]

I suggest let's not change standards so easily. The library.properties file has till now not required escaping of colons. Of course we can decide to change that based on reasons, such as a java .properties standard, but let's take some time to think about it. Does this file need to be a java standard properties file? Is user readability important?