Merge bitcoin/bitcoin#33781: clang-tidy: Remove no longer needed NOLINT

038849e clang-tidy: Remove no longer needed NOLINT (Hennadii Stepanov)

Pull request description:

  From https://github.com/bitcoin/bitcoin/pull/33714/files#r2491476516:
  > Actually, the `NOLINT` was fixed and can be removed? You've confirmed that it is undeclared on the listed platforms, so it can't be hit by `readability-redundant-declaration`

ACKs for top commit:
  maflcko:
    lgtm ACK 038849e
  l0rinc:
    I wanted to ask the same on the original PR but forgot - ACK 038849e

Tree-SHA512: c0b24235425e80baeac3158c7169122364f31140367bc289430d34f01cd38f9f6a3931319f6fe4e1dc86bc4d87e21a5b4b8a2263c199e8083593f89ce592a177

Author: fanquake

src/randomenv.cpp

@@ -62,7 +62,7 @@
     defined(__NetBSD__) || \
     defined(__OpenBSD__) || \
     defined(__illumos__)
-extern char** environ; // NOLINT(readability-redundant-declaration): Necessary on the above platforms
+extern char** environ; // Necessary on the above platforms
 #endif
 
 namespace {