Differential D18222

lint: fix confusing comments in test vector file
ClosedPublic
Actions

Authored by PiRK on Jun 7 2025, 10:08.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABCe8c37660b89c: lint: fix confusing comments in test vector file

Summary

The --skip-arguments argument was removed in D2882, replaced by a FUNCTION_NAMES_AND_NUMBER_OF_LEADING_ARGUMENTS list.

Depends on D18221

Test Plan

python -m doctest test/lint/lint-format-strings.py

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 33571
Build 66621: Build Diff	build-without-wallet · lint-circular-dependencies · build-diff · build-clang-tidy · build-clang · build-debug
Build 66620: arc lint + arc unit

Event Timeline

PiRK created this revision.Jun 7 2025, 10:08

Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 7 2025, 10:08

PiRK requested review of this revision.Jun 7 2025, 10:08

Harbormaster completed remote builds in B33571: Diff 54395.Jun 7 2025, 10:22

PiRK added a child revision: D18223: init: Error if ignored bitcoin.conf file is found.Jun 7 2025, 15:09

Fabien accepted this revision.Jun 9 2025, 09:30

This revision is now accepted and ready to land.Jun 9 2025, 09:30

Closed by commit rABCe8c37660b89c: lint: fix confusing comments in test vector file (authored by PiRK). · Explain WhyJun 9 2025, 09:43

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABCe8c37660b89c: lint: fix confusing comments in test vector file.

Revision Contents
Changeset List

Path

Size

test/

lint/

lint-format-strings-tests-skip-arguments.txt

6 lines

Diff 54395

View Options

test/lint/lint-format-strings-tests-skip-arguments.txt

lint: fix confusing comments in test vector fileClosedPublicActions