ci, iwyu: Double maximum line length for includes #27707

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:230520-tidy changing 1 files +1 −0
  1. hebasto commented at 12:21 pm on May 20, 2023: member

    This PR makes the IWYU output in the CI ’tidy’ task more useful by avoiding most cases where a comment ends with an ellipsis like that:

    0#include "primitives/transaction.h"  // for CTxIn, CMutableTransaction, CTra...
    
  2. ci, iwyu: Double maximum line length for includes 98ea798411
  3. DrahtBot commented at 12:21 pm on May 20, 2023: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK TheCharlatan

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #27710 (ci, iwyu: Update mappings by hebasto)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  4. TheCharlatan approved
  5. TheCharlatan commented at 7:33 pm on May 20, 2023: contributor
    ACK 98ea79841177cf7492ec1de1ba1979f75390c63b
  6. maflcko approved
  7. maflcko commented at 6:20 am on May 22, 2023: member
    lgtm
  8. fanquake merged this on May 22, 2023
  9. fanquake closed this on May 22, 2023

  10. hebasto deleted the branch on May 22, 2023
  11. sidhujag referenced this in commit 1bd30cb21f on May 23, 2023
  12. bitcoin locked this on May 21, 2024

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-10-30 00:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me