Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename Whitelist to Allowlist #207

Merged
merged 3 commits into from
Aug 21, 2024
Merged

Rename Whitelist to Allowlist #207

merged 3 commits into from
Aug 21, 2024

Conversation

ezdac
Copy link

@ezdac ezdac commented Aug 21, 2024

Splitting this one up, as it is easy to review and reduces the diff from another PR.

@ezdac ezdac requested a review from palango August 21, 2024 09:24
Copy link

@palango palango left a comment

Choose a reason for hiding this comment

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

Need to fix some comments as well

core/celo_multi_gaspool_test.go Outdated Show resolved Hide resolved
common/celo_types_test.go Outdated Show resolved Hide resolved
core/state_transition.go Outdated Show resolved Hide resolved
core/state_transition.go Outdated Show resolved Hide resolved
core/txpool/legacypool/celo_legacypool.go Outdated Show resolved Hide resolved
core/txpool/legacypool/celo_legacypool.go Outdated Show resolved Hide resolved
@ezdac ezdac merged commit 877ff8f into celo7 Aug 21, 2024
7 checks passed
@ezdac ezdac deleted the ezdac/allowlist branch August 21, 2024 13:29
karlb pushed a commit that referenced this pull request Aug 30, 2024
* Rename whitelist to allowlist

* Rename whitelist to allowlist in comments and variables

Co-authored-by: Paul Lange <[email protected]>

* Change 'non-whitelisted' to 'unregistered' in public error msg

---------

Co-authored-by: Paul Lange <[email protected]>
karlb pushed a commit that referenced this pull request Oct 11, 2024
* Rename whitelist to allowlist

* Rename whitelist to allowlist in comments and variables

Co-authored-by: Paul Lange <[email protected]>

* Change 'non-whitelisted' to 'unregistered' in public error msg

---------

Co-authored-by: Paul Lange <[email protected]>
piersy pushed a commit that referenced this pull request Oct 15, 2024
* Rename whitelist to allowlist

* Rename whitelist to allowlist in comments and variables

Co-authored-by: Paul Lange <[email protected]>

* Change 'non-whitelisted' to 'unregistered' in public error msg

---------

Co-authored-by: Paul Lange <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants