BREAKING(yaml): rename ParseOptions.noArrayIndent to ParseOptions.arrayIndent

[iuioiua]

What's changed

The ParseOptions.noArrayIndent option has been renamed to ParseOptions.arrayIndent. As a result, the polarity of the option has been swapped. In other words, true behavior has been changed to false behavior, and visa-versa. This only affects users who currently use ParseOptions.noArrayIndent.

Motivation

This change makes the option far easier to understand by avoiding a negative-tensed name.

Migration guide

Use ParseOptions.arrayIndent instead of ParseOptions.noArrayIndent and swap the polarity of the option.

import { parse } from "@std/yaml/parse";

const yamlString = `fruits:
  - Apple
  - Banana
  - Cherry`;

- parse(yamlString, { noArrayIndent: false });
+ parse(yamlString, { arrayIndent: true });

Related

Towards #5124

[codecov]

Codecov Report

Attention: Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 95.71%. Comparing base (f1952ec) to head (529b8d4).

Files	Patch %	Lines
yaml/_dumper.ts	75.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #5286   +/-   ##
=======================================
  Coverage   95.71%   95.71%           
=======================================
  Files         457      457           
  Lines       37955    37955           
  Branches     5560     5560           
=======================================
  Hits        36330    36330           
  Misses       1584     1584           
  Partials       41       41           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[kt3k]

LGTM