Repository navigation
feat: add headless option to build() - #172
Conversation
Adds a 'headless' option to BuildOptions that, when set to true, omits the XML declaration and DOCTYPE from the output. This is useful for embedding plist fragments. Works in both the Node.js (xmlbuilder) and browser (native) code paths. Supersedes #159
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
🦋 Changeset detectedLatest commit: 1c4d557 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
This approach seems overly restrictive, as it only exposes the headless xmlbuilder create option. It’s also unclear what problem this is intended to solve. The advantage of #159 is that it supports all |
|
@dnicolson did you see this comment? |
|
@TooTallNate I did, but it was largely contradicted by the next comment. |
|
This |
Adds a
headlessoption toBuildOptionsthat omits the XML declaration (<?xml ...?>) and DOCTYPE from the output. Useful for embedding plist fragments.Works in both Node.js (
xmlbuilder) and browser (native) code paths.Supersedes #159 — implements the alternative API suggested in the code review comments (implementation-agnostic
BuildOptionsproperty instead of exposingxmlbuilder-specific types).Usage