@epage Would you mind taking a look at this fix for #314 since you filed that issue? Big thanks in advance.

Looks good. Thanks!

Big thanks for taking a look.

(I will move out the CHANGELOG.md changes into a separate PR to avoid conflicts with other upcoming PRs.)

Can you change this to a deprecation of the old name? I know it's a major version bump so we can just remove it, but this is the biggest API change and deprecations are way friendlier then renames given that the compile error can directly tell the user how to rename their call. It'll make upgrading to the new major version much smoother.

Maybe the warning can even be like "highlight was renamed to highlight_line to make it clear it should be passed a single line at a time"

Sure, no problem at all. I have pushed a commit to fix that.

Here is how the deprecation message looks in the wild. (Generated by temporarily using the deprecated variant in our own example code.)

warning: use of deprecated associated function `syntect::easy::HighlightLines::<'a>::highlight`: Renamed to `highlight_line` to make it clear it should be passed a single line at a time
   --> examples/syncat.rs:106:83
    |
106 |                     let regions: Vec<(Style, &str)> = highlighter.highlight_lines.highlight(&line, &ss);
    |                                                                                   ^^^^^^^^^
    |
    = note: `#[warn(deprecated)]` on by default