Itin

[SzymonDrosdzol]

No description provided.

[v1ktor0t]

the other thing i was thinking was moving the requirement for the value to start with 9 in the regex. but i think having everything encapsulated in a single function ultimately ends up working better.

[v1ktor0t]

any reason why it's not VeryLikely in this case?

[SzymonDrosdzol]

That's how they wanted it for SSN, so I'm assuming it'll be the same here.
See: https://github.com/google/osv-scalibr/pull/2208/changes#diff-de016d11e0488589101a53de1c99491fe882d6bd27d4b4bb5f3666cb9966c728R76

I think the idea is VeryLikely is reserved for checksum validated stuff like CCs and IBANs

[SzymonDrosdzol]

the other thing i was thinking was moving the requirement for the value to start with 9 in the regex. but i think having everything encapsulated in a single function ultimately ends up working better.

I think this way is far more readable and less error prone. Though I see why some might prefer it in the regex.

[v1ktor0t]

lgtm 🚀