expand tinkr to reading metadata in toml?

[maelle]

If the first line is "---" (or more I guess) it is YAML, if it is "+++" it is TOML.

The package wouldn't need to parse TOML, just to keep it as text to not break it.

Happy to make a PR.

@yabellini

[zkamvar]

Oh that's a good and simple addition! Go for it!

[maelle]

Should the change be a bit more ambitious:

• renaming the current "yaml" to "front matter". It could be YAML, TOML, JSON, maybe something else.
• adding a slot, or something else, to indicate the format of the front matter.
• somehow deprecate the former "yaml" name, emitting a message if it is accessed. This reminds me of Deprecate components of output lists igraph/rigraph#1576

[zkamvar]

I think that's a great idea and R6 makes it really easy to do this with active bindings.

tink <- R6::R6Class("tink",
   public = list(
     body = NULL,
     front = "matter"
   ),
   active = list(
     yaml = function(yml) {
       message("oh no, we do not use this")
       if (!missing(yml)) self$front <- yml 
       self$front
     }
   )
 )

 n <- tink$new()
 n$yaml <- "---\nhey: there\n---"
#> oh no, we do not use this
 n$front
#> [1] "---\nhey: there\n---"
 n$yaml
#> oh no, we do not use this
#> [1] "---\nhey: there\n---"

^{Created on 2024-12-13 with reprex v2.1.1}

[maelle]

Ok, thank you, I hope to work on this soon then! Or... next year 😸