refactor(yaml): remove tag property

[timreichen]

Changes
This PR removes the tag property in DumperState.

Reasoning
tag is only used in writeBlockMapping(), so it can be passed as an argument instead.

[codecov]

Codecov Report

Attention: Patch coverage is 86.66667% with 2 lines in your changes missing coverage. Please review.

Project coverage is 96.28%. Comparing base (8cb8496) to head (de09e54).
Report is 1 commits behind head on main.

Files	Patch %	Lines
yaml/_dumper_state.ts	86.66%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5689      +/-   ##
==========================================
- Coverage   96.28%   96.28%   -0.01%     
==========================================
  Files         474      474              
  Lines       38392    38381      -11     
  Branches     5576     5576              
==========================================
- Hits        36967    36956      -11     
  Misses       1383     1383              
  Partials       42       42              

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

[iuioiua]

Can you please only include changes that directly pertain to removing the tag property? We can handle any other refactors in a separate PR.

[timreichen]

Can you please only include changes that directly pertain to removing the tag property? We can handle any other refactors in a separate PR.

Alright, I undo the detectType changes

[iuioiua]

The return type for this method has changed from explicitly boolean to now being implicitly string | null. Why? Return types should be explicit and not change.

[timreichen]

Since the tag property is removed and was set inside this function, it has to be returned instead. It returns string(former true) and null (former false).

[iuioiua]

LGTM. Thank you.