[clang-tidy] Check request: avoid dangerous ctad constructing container from iterators pair

[denzor200]

Assume we have a vector of 4 integers:

std::vector<int> v = {1,2,3,4};

And then imagine we need to make a copy of that vector:

auto v2 = std::vector{v.begin(), v.end()};

Are you expecting v2 to be a vector of 4 integers? Surprise :) This is a std::vector<std::vector<int>::iterator> with 2 iterators.
The way to fix that copying:

auto v2 = std::vector(v.begin(), v.end());

Note that it's easy to reach that typo due to code styles which are forcing programmers to always use auto and always prefer braced initialization.
You may suggest to change copying to auto v2 = v;, that's correct but not always possible - imagine we are copying subarray.