Skip to content

add func to plistyamlplist_lib so external projects can import easily - #18

Open
discentem wants to merge 1 commit into
grahampugh:devfrom
discentem:main
Open

discentem wants to merge 1 commit into
grahampugh:devfrom
discentem:main

Conversation

@discentem

@discentem discentem commented Apr 17, 2025

Copy link
Copy Markdown

This change makes it easier to use plist-yaml-plist as a library by giving callers back a string instead of writing automatically writing to a file. I already tested this change in my fork though please test yourself in case I am missing something. I am also happy to add a unit test if desired.

https://github.com/autopkg/discentem-recipes/blob/main/SharedProcessors/lib/plist_yaml_plist/plist_yaml.py#L109C1-L121C18

… conversion machinery

This change makes it easier to use plist-yaml-plist as a library by giving callers a string instead of being forced to immediately write a file to disk.
@grahampugh
grahampugh changed the base branch from main to dev October 6, 2025 19:45
@grahampugh

Copy link
Copy Markdown
Owner

Hi, can you take a look at the dev branch, it's a merge of fairly significant changes, so I'm not sure if your issue still stands or how I would be able to merge it in if so.

@discentem

Copy link
Copy Markdown
Author

Hi, can you take a look at the dev branch, it's a merge of fairly significant changes, so I'm not sure if your issue still stands or how I would be able to merge it in if so.

Should I rebase on dev to see if the changes are still relevant?

@grahampugh

Copy link
Copy Markdown
Owner

Yes please, if you can. It's different enough that I can't figure out what the original intention was.

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