Should the register function add a lang key to _quarto.yml?

[Nenuial]

Following @joelnitta in #66:

Why is the last lang:en needed? Isn't that redundant, considering that we already have mainlanguage: 'en'?

babelquarto/R/register.R

Line 29 in c0804a4

sprintf("lang: %s", main_language)

I wonder if we should really "force" this option on the user. It isn't redundant as I explained in #66 but it also isn't absolutely necessary and I believe it should remain the choice of the user whether to have this in _quarto.yml

@maelle what do you think?

[maelle]

So, recently in a book where I used babelquarto, I used quarto::quarto_render() to ensure a fix I had done worked for the main language before I added it to other languages. Now my book's main language was English, but I still feel it could be useful and does not hurt?

Feel free to disagree!

[Nenuial]

it could be useful and does not hurt?

I agree that it's useful. It can however hurt if you start from an existing project and already have a lang: en (or other language) in your config and then add the main language using register_main_language().

I'd therefore suggest checking if lang already exists and if there is no lang key, then we add it.
I've made a PR that would do that.

[maelle]

thank you!!