Skip to content

use javaproperties library to parse #77

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 16, 2025

Conversation

mingness
Copy link
Collaborator

This PR addresses #76 by using a library specifically designed to treat java property files in python.
This means all escaped characters will be dealt with properly.

This was tested with the VJ Motion library, which does have an escaped colon in the value of url.

the output for the script is below, showing no escaped colons in the values.

properties dict: {'type': 'library', 'source': 'https://github.com/vincentsijben/vjmotion-processing/releases/latest/download/VJMotion.txt', 'name': 'VJ Motion', 'authors': 'Vincent Sijben', 'url': 'https://vincentsijben.github.io/vjmotion-processing/', 'categories': 'Sound,Utilities,Math', 'sentence': 'A collection of helper functions to create live, audio-reactive visuals with BPM-synced animation, Arduino input and frequency analyzing.', 'paragraph': '', 'version': 8, 'prettyVersion': '0.0.8', 'minRevision': 0, 'maxRevision': 0}
T

Copy link

netlify bot commented May 16, 2025

Deploy Preview for processing-contributions ready!

Name Link
🔨 Latest commit 968db63
🔍 Latest deploy log https://app.netlify.com/projects/processing-contributions/deploys/682781c664bad4000803ed14
😎 Deploy Preview https://deploy-preview-77--processing-contributions.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@mingness mingness linked an issue May 16, 2025 that may be closed by this pull request
@mingness mingness requested review from SableRaf and Stefterv May 16, 2025 18:20
@mingness
Copy link
Collaborator Author

apologies, I'm on the east coast, so this will come to you rather late in your day. Of course, please ignore till you have time. This is mainly to notify you of the work. I will likely merge later today.

@mingness mingness merged commit 513c92c into processing:main May 16, 2025
4 checks passed
@mingness mingness deleted the treat-java-properties branch May 16, 2025 21:14
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.

Escaped : in .properties causes invalid URLs in generated .yaml
1 participant