Skip to content

Conversation

@glozow
Copy link
Member

@glozow glozow commented Feb 28, 2023

Since 5497c14, verify-commits.py uses git merge-tree which requires git v2.38 or later. Fix the lint jobs on master (e.g. https://cirrus-ci.com/task/4971007513985024).

@glozow glozow requested a review from achow101 February 28, 2023 11:39
@DrahtBot
Copy link
Contributor

DrahtBot commented Feb 28, 2023

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 achow101, hebasto
Stale ACK MarcoFalke

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

Copy link
Member

@hebasto hebasto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 836720e366f61f23c7a56775e11d58890b002d75

@glozow glozow force-pushed the 2023-03-lint-bump-git-lunar branch from 836720e to 257dfef Compare February 28, 2023 14:42
Since 5497c14, verify-commits.py requires git merge-tree which is only
available in git v2.38 or later.
@maflcko
Copy link
Member

maflcko commented Feb 28, 2023

lgtm ACK 257dfef897db0ece387511d0592076381aaeff2b

The title and commit message can be adjusted to ci: ..., if you retouch

@glozow glozow force-pushed the 2023-03-lint-bump-git-lunar branch from 257dfef to a984bee Compare February 28, 2023 14:43
@DrahtBot DrahtBot requested a review from hebasto February 28, 2023 14:43
@glozow
Copy link
Member Author

glozow commented Feb 28, 2023

lgtm ACK 257dfef

sorry, pushed again because commit message still said lunar

@glozow glozow changed the title [ci] bump lint task to lunar for git v2.38 ci: bump lint task to lunar for git v2.38 Feb 28, 2023
@DrahtBot DrahtBot added the Tests label Feb 28, 2023
@glozow glozow changed the title ci: bump lint task to lunar for git v2.38 ci: bump lint task to bookworm for git v2.38 Feb 28, 2023
@achow101
Copy link
Member

ACK a984bee

@DrahtBot DrahtBot requested a review from maflcko February 28, 2023 15:04
Copy link
Member

@hebasto hebasto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

re-ACK a984bee

@DrahtBot DrahtBot requested a review from maflcko February 28, 2023 15:34
@fanquake fanquake merged commit 9384536 into bitcoin:master Feb 28, 2023
@glozow glozow deleted the 2023-03-lint-bump-git-lunar branch February 28, 2023 16:02
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Mar 1, 2023
a984bee [ci] change lint to bookworm for git v2.38 (glozow)

Pull request description:

  Since 5497c14, verify-commits.py uses `git merge-tree` which requires git v2.38 or later. Fix the lint jobs on master (e.g. https://cirrus-ci.com/task/4971007513985024).

ACKs for top commit:
  achow101:
    ACK a984bee
  hebasto:
    re-ACK a984bee

Tree-SHA512: dd50598aefb6f9c86cf221dea27fcc521e335cb182f7a3abcb3215d3991794354085be3e07c133ab74ca8b957e3a6acc43722899165957b3d898867d6253ebdc
@bitcoin bitcoin locked and limited conversation to collaborators Feb 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants