Looks like wrong release notes file was copied.

Feedback + reworking some comments

Why not use the registerVotes and addToReconcile API like the other tests, rather than massaging the internals ?

• Address feedback
• Restrict filtering to finalized items of any type and invalidated blocks only. See comments for rationale.

fixup f-string

D13219 is a draft to give an idea what the followup can look like to generalize this. Still needs updates to tests and renaming pending acceptance of this diff.

Use a bloom filter instead

• Rebase and fixup release notes
• Add miner fund rejection test case

Does not matter now that this patch has been reviewed a couple times but lots of little things could have been their own diffs like making some things const, moving nNoLimit, etc.

Fix the callsite instead of modifying the implementation

More test cleanup

Rebase and add null check

add release note

• Remove unnecessary fParked
• Add new functional test to ensure miner fund address and amount can be changed by policy

feedback

feedback

Rebase & fixup according to feedback

feedback

Cleanup unnecessary check against getbestblockhash

Incorporate nits except moving checkEarlyBlocks. This would work as the test is now but it is not clear to me that this actually adds value.

• Fix leftover assumptions from previous iteration. Genesis block is now robustly supported.
• Rename pindex -> blockIndex
• Improve test coverage for early blocks including genesis
• Refactor tests to make better use of lambdas and fix other minor issues throughout.

Improve file names. Remove now unneeded GetParkingPoliciesForBlock and other cleanup.

Address all feedback. Add more test cases and cleanup test code.

first pass