Skip to content

Conversation

@meshcollider
Copy link
Contributor

Suggested here as a trivial follow-up: #23123 (comment)

Makes RESCAN_REQUIRED consistent with NEED_REWRITE

@achow101
Copy link
Member

scripted diff?

-BEGIN VERIFY SCRIPT-
git grep -l 'RESCAN_REQUIRED' src | xargs sed -i 's/RESCAN_REQUIRED/NEED_RESCAN/g'
-END VERIFY SCRIPT-
@meshcollider
Copy link
Contributor Author

Added scripted diff

Copy link
Member

@jonatack jonatack left a comment

Choose a reason for hiding this comment

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

ACK 8615507

LOAD_FAIL,
NEED_REWRITE,
RESCAN_REQUIRED
NEED_RESCAN
Copy link
Member

Choose a reason for hiding this comment

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

if you have to retouch

Suggested change
NEED_RESCAN
NEED_RESCAN,

@achow101
Copy link
Member

ACK 8615507

@fanquake fanquake changed the title trivial refactor: rename DBErrors::RESCAN_REQUIRED to NEED_RESCAN refactor: rename DBErrors::RESCAN_REQUIRED to NEED_RESCAN Sep 30, 2021
fanquake added a commit that referenced this pull request Oct 1, 2021
8615507 scripted-diff: rename DBErrors::RESCAN_REQUIRED to NEED_RESCAN (Samuel Dobson)

Pull request description:

  Suggested here as a trivial follow-up: #23123 (comment)

  Makes RESCAN_REQUIRED consistent with NEED_REWRITE

ACKs for top commit:
  achow101:
    ACK 8615507
  jonatack:
    ACK 8615507

Tree-SHA512: 82d057c45e192cd6dd8a47675b52699e6cbc82272609a971e6e5d6796aad14a941a70e40d3913dbb611f79c8eadff8030c60ea6f203f2edc3720c0e78c166b97
@fanquake
Copy link
Member

fanquake commented Oct 1, 2021

This has been merged.

@fanquake fanquake closed this Oct 1, 2021
@DrahtBot
Copy link
Contributor

DrahtBot commented Oct 1, 2021

🐙 This pull request conflicts with the target branch and needs rebase.

Want to unsubscribe from rebase notifications on this pull request? Just convert this pull request to a "draft".

@meshcollider meshcollider deleted the 202110_rename_rescan_enum branch October 1, 2021 03:51
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Oct 1, 2021
…ED_RESCAN

8615507 scripted-diff: rename DBErrors::RESCAN_REQUIRED to NEED_RESCAN (Samuel Dobson)

Pull request description:

  Suggested here as a trivial follow-up: bitcoin#23123 (comment)

  Makes RESCAN_REQUIRED consistent with NEED_REWRITE

ACKs for top commit:
  achow101:
    ACK 8615507
  jonatack:
    ACK 8615507

Tree-SHA512: 82d057c45e192cd6dd8a47675b52699e6cbc82272609a971e6e5d6796aad14a941a70e40d3913dbb611f79c8eadff8030c60ea6f203f2edc3720c0e78c166b97
@bitcoin bitcoin locked and limited conversation to collaborators Oct 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants