Skip to content

Conversation

maflcko
Copy link
Member

@maflcko maflcko commented Oct 5, 2022

clang-format could be used in scripted diffs, but remained largely unused.

So remove the install bloat, as it is unlikely to be used in the future.

clang-format could be used in scripted diffs, but remained largely
unused.
@DrahtBot DrahtBot added the Tests label Oct 5, 2022
@fanquake
Copy link
Member

fanquake commented Oct 5, 2022

ACK fa04376

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 fa04376

@maflcko maflcko merged commit b2e6d37 into bitcoin:master Oct 5, 2022
@maflcko maflcko deleted the 2210-ci-less-lint-🕚 branch October 5, 2022 09:56
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Oct 5, 2022
fa04376 Remove clang-format from lint task (MacroFake)

Pull request description:

  clang-format could be used in scripted diffs, but remained largely unused.

  So remove the install bloat, as it is unlikely to be used in the future.

ACKs for top commit:
  fanquake:
    ACK fa04376
  hebasto:
    ACK fa04376

Tree-SHA512: e0a3ee47d2aa2565dd34676914c558c985eaeb522a05f10bcaac115871edcf0d7f101b517e4d452ca5223c40b18ad02883c31e2da3d1f4ff86464a9af0097b11
@bitcoin bitcoin locked and limited conversation to collaborators Oct 5, 2023
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.

4 participants