<table border="1" cellspacing="0" cellpadding="8">
    <tr>
        <th>Issue</th>
        <td>
            <a href=https://github.com/llvm/llvm-project/issues/64049>64049</a>
        </td>
    </tr>

    <tr>
        <th>Summary</th>
        <td>
            Deprecate key-value syntax in CheckOptions
        </td>
    </tr>

    <tr>
      <th>Labels</th>
      <td>
            clang-tidy,
            code-cleanup,
            code-quality
      </td>
    </tr>

    <tr>
      <th>Assignees</th>
      <td>
      </td>
    </tr>

    <tr>
      <th>Reporter</th>
      <td>
          carlosgalvezp
      </td>
    </tr>
</table>

<pre>
    Currently, clang-tidy supports specifying `CheckOptions` in two ways:

* Old way, which has been removed from the documentation:

```
CheckOptions: [
{ key: x, value: y},
]
´´´

* New way, showing in the documentation:
```
Check Options: 
  x: y
```

The source code handling the YAML parsing still supports the old way, for backwards compatibility reasons. This makes the code more complicated than it needs to be to support a no longer documented feature. Complex code is hard to understand, maintain and extend.

Therefore, it would be good to deprecate the old key-value format to be able to clean the YAML parser. We could print a warning so people get informed, and even create a tool that automatically converts a .clang-tidy file from the old format into the new format to save people time, easing the transition.
</pre>
<img width="1px" height="1px" alt="" src="http://email.email.llvm.org/o/eJx0lMGOq7gShp8GNqVGYEJyWbDoTpTVnTmbI41mWeAKeGJsxi6SME8_KpLudLfOSJEDhf3XV3_ZxhhN74iapHpLqkOKMw8-NB0G62OP9kL_TGnr9dLs5xDIsV0StYfOoutf2OgF4jxNPnCEOFFnTotxPSTbfD9Qd_4xsfEuJtscjAO-erjiEpPyNckPSf4-qlf4YbV8EunrYLoBBozQEjkINPoLaTgFPwIPBNp380iOUaS_S23zx299_cJQvoLUeJ-3e4MzLRK7Sc4L2pnkbUl2h0TtH7Oqw-Nhr5K3zZfxG__vdH3nj4O_igdS8H_j_gIUPpOuYYDbHepXa-7jz4Eg-jl0BJ3XBAM6bSW95P7z9bf_w4QhSiCysfbZLfnun66ffIAWu_MVg47Q-XFCNq2xhhcIhNG7mMHPwUQY8Uz35WvC0Qda51vTIZMGHtCBYXBEOgJ7aEnGR2JAcB6sdz2FD2-kvYQ8B8pgL1J0u4ubCAMGLetnpylERqcFd0TjGI0DdBroxuR09s2WQCcfSCYbhqufrRaS3vtVTtMUSIA_jDjT8rLuA_FiRH6gY2tX_s4Suq-uUsjgDyletKdgnFR3xeBWuz1M5CdL0BODcSJKK_uKfCEHXSABQGDvrfjGgDP7Edl0aO0CnXcXkmYhZJ9O3MlYep4HYX8QG8d-jTm6fqoi4oXeYdiMqyeE8X2XcEAXjWy9LNVNqeuyxpSaYlvnxUblRZ0OzQ6xrcpTVfyvVqddXRa7bdvuiiLXp11bF3VqGpWrMt-psqirsqyzalO1Jyorvem2xVZtk01OIxqbWXsZMx_61MQ4U7Pd5Js6tdiSjeslpNSz1EQpOY1KyW54WVswT1-Df88om1SC1SENjci_tHMfk01uTeT4TMiGLTWHj84_Ox4Xx3iTI_v5ykjnYJuBeVpvLHVM1LE3PMxt1vkxUUcRfvy9TMH_RR0n6riWFRN1XCv7NwAA__9nXcMp">