ci: Properly set COMMIT_RANGE in lint task #26791

pull maflcko wants to merge 1 commits into bitcoin:master from maflcko:2301-ci-lint-🥗 changing 2 files +2 −2
  1. maflcko commented at 1:13 PM on January 2, 2023: member

    Currently the variable holds (apart from the commits in the pull request) all commits to master since the pull was opened.

    This is problematic, because already merged commits are linted in unrelated pulls, leading to:

    Now that the CI has the /merge commit (since commit fad7281d7842f337932cf44e703fdd631230ddd6), COMMIT_RANGE can simply be set to HEAD~..HEAD to only hold the changes in the pull.

  2. ci: Properly set COMMIT_RANGE in lint task fa5cbf2290
  3. DrahtBot commented at 1:13 PM on January 2, 2023: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

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

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK fanquake

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

  4. DrahtBot added the label Tests on Jan 2, 2023
  5. fanquake approved
  6. fanquake commented at 11:55 AM on January 4, 2023: member

    ACK fa5cbf2290dab08296cabbd973e9adefc883f3fc

  7. maflcko merged this on Jan 4, 2023
  8. maflcko closed this on Jan 4, 2023

  9. maflcko deleted the branch on Jan 4, 2023
  10. sidhujag referenced this in commit 66028ebdb8 on Jan 4, 2023
  11. bitcoin locked this on Jan 4, 2024
Labels

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: 2026-04-24 09:14 UTC

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