feat: change signature of `convert`

[xitep]

What is the proposed feature?

I suggest to change the signature of convert (in html-to-markdown-rs) from

fn convert(html: &str, options: Option<ConversionOptions>)

to

fn convert(html: &str, options: ConversionOptions)

Why would this be a good addition?

I don't see much much point in the Option; callers can simply supply Default::default() (or ConversionOptions::default() for None. The API becomes a bit more explicit and easier to read (no wondering what a None parameter does that might be different from the default) while reducing the need to wrap the options with Some.

[xitep]

thanks @Goldziher, i dunno what happened, but my comment on the completed patch disappeared. i wanted to add that you could avoid the cost of monomorphisation by extracting the generic parameter into a separate small method, like in:

pub fn convert(..., options: impl Into<...>) -> Result<..> {
   convert_(..., options.into().unwrap_or_default())
}

fn convert_(html: &str, options: ConversionOptions) -> Result<..> {
 ...
}