I don't think we need to maintain a linter for reintroducing boost::bind at this point.
lint: remove boost::bind lint #24785
pull fanquake wants to merge 1 commits into bitcoin:master from fanquake:remove_boost_bind_lint changing 1 files +0 −21-
fanquake commented at 1:11 PM on April 6, 2022: member
-
4105a54381
lint: remove boost::bind linter
I don't think we need to maintain a linter for reintroducing boost::bind at this point.
- fanquake added the label Tests on Apr 6, 2022
- hebasto approved
-
hebasto commented at 1:18 PM on April 6, 2022: member
ACK 4105a543817c47bda21739d5ad3269677f84e861, I have reviewed the code and it looks OK, I agree it can be merged.
The
boost/bind/bind.hppheader is even not allowed bylint-includes.sh. - MarcoFalke merged this on Apr 6, 2022
- MarcoFalke closed this on Apr 6, 2022
- fanquake deleted the branch on Apr 6, 2022
- sidhujag referenced this in commit 9642b7b2ce on Apr 6, 2022
- PastaPastaPasta referenced this in commit dc4a2ec7fa on Apr 7, 2022
- PastaPastaPasta referenced this in commit 5e2d4d5c19 on Apr 7, 2022
- UdjinM6 referenced this in commit 671d3ec1cf on Apr 7, 2022
- DrahtBot locked this on Apr 6, 2023