Skip to content

add options to keep metadata from either old or new file or none of them#54

Merged
davidar merged 2 commits intodavidar:masterfrom
marvinjuette:add-option-for-yaml-header
May 10, 2025
Merged

add options to keep metadata from either old or new file or none of them#54
davidar merged 2 commits intodavidar:masterfrom
marvinjuette:add-option-for-yaml-header

Conversation

@marvinjuette
Copy link
Copy Markdown
Contributor

@marvinjuette marvinjuette commented Apr 17, 2025

As the title says. Should solve #10

Open for feedback, either code wise or on the general approach.

@marvinjuette
Copy link
Copy Markdown
Contributor Author

marvinjuette commented May 2, 2025

Hej @davidar,

just wanted to check in if there are any updates/issues on your side?

@davidar
Copy link
Copy Markdown
Owner

davidar commented May 4, 2025

It doesn't seem to be working for me... can you add a test case to index.spec.ts?

@marvinjuette marvinjuette force-pushed the add-option-for-yaml-header branch from d9154e9 to 9d228c1 Compare May 5, 2025 22:02
@marvinjuette marvinjuette force-pushed the add-option-for-yaml-header branch from 9d228c1 to 958fdc6 Compare May 5, 2025 22:07
@marvinjuette marvinjuette force-pushed the add-option-for-yaml-header branch from 958fdc6 to ffab700 Compare May 5, 2025 22:18
@marvinjuette
Copy link
Copy Markdown
Contributor Author

@davidar I added some tests and I also enable the standalone option when the --metadata option is provided, because without it pandoc ignores the metadata inside the document from what I understand.

@davidar davidar linked an issue May 10, 2025 that may be closed by this pull request
@davidar davidar merged commit 549b583 into davidar:master May 10, 2025
4 checks passed
@davidar
Copy link
Copy Markdown
Owner

davidar commented May 10, 2025

Thanks!

@marvinjuette marvinjuette deleted the add-option-for-yaml-header branch May 10, 2025 11:16
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.

YAML meta data ignored

3 participants