test: Remove feature_txindex_compatibility.py #28753

pull BrandonOdiwuor wants to merge 1 commits into bitcoin:master from BrandonOdiwuor:remove_feature_txindex_compatibility_py changing 2 files +0 −67

BrandonOdiwuor commented at 10:22 AM on October 30, 2023: contributor

Fixes #28421, see #28195 (comment)

Remove feature_txindex_compatibility.py in V27, follow up to #28195 being merged which is included in v26
DrahtBot commented at 10:22 AM on October 30, 2023: contributor


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



Code Coverage

For detailed information about the code coverage, see the test coverage report.



Reviews

See the guideline for information on the review process.

Type Reviewers

ACK maflcko, theStack, stickies-v

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:
- #28052 (blockstorage: XOR blocksdir *.dat files by maflcko)
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.
maflcko commented at 10:26 AM on October 30, 2023: member

I think you can remove the [27.x] prefix. Branch-off has already happened, so this should be clear.
BrandonOdiwuor renamed this:
~~[27.x] test: Remove feature_txindex_compatibility.py in V27~~
test: Remove feature_txindex_compatibility.py in V27
on Oct 30, 2023
DrahtBot added the label Tests on Oct 30, 2023
Remove feature_txindex_compatibility.py in V27 897d6dd42b
BrandonOdiwuor force-pushed on Oct 30, 2023
DrahtBot added the label CI failed on Oct 30, 2023
fanquake renamed this:
~~test: Remove feature_txindex_compatibility.py in V27~~
test: Remove feature_txindex_compatibility.py
on Oct 30, 2023
maflcko commented at 10:38 AM on October 30, 2023: member

lgtm ACK 897d6dd42b78ab8569cbd7ae5e27f277b9acaeff
theStack approved
theStack commented at 10:54 AM on October 30, 2023: contributor

ACK 897d6dd42b78ab8569cbd7ae5e27f277b9acaeff
DrahtBot removed the label CI failed on Oct 30, 2023
stickies-v approved
stickies-v commented at 12:50 PM on October 30, 2023: contributor

ACK 897d6dd42b78ab8569cbd7ae5e27f277b9acaeff
fanquake merged this on Oct 30, 2023
fanquake closed this on Oct 30, 2023
BrandonOdiwuor deleted the branch on Nov 21, 2023
bitcoin locked this on Nov 20, 2024