Skip to content

Set PSE excluded addresses inside v6 upgrade#98

Merged
ysv merged 3 commits into
release-v6from
yaroslav/release-v6-pse-excluded-addresses-2
Mar 3, 2026
Merged

Set PSE excluded addresses inside v6 upgrade#98
ysv merged 3 commits into
release-v6from
yaroslav/release-v6-pse-excluded-addresses-2

Conversation

@ysv

@ysv ysv commented Mar 2, 2026

Copy link
Copy Markdown
Contributor

Description

Reviewers checklist:

  • Try to write more meaningful comments with clear actions to be taken.
  • Nit-picking should be unblocking. Focus on core issues.

Authors checklist

  • Provide a concise and meaningful description
  • Review the code yourself first, before making the PR.
  • Annotate your PR in places that require explanation.
  • Think and try to split the PR to smaller PR if it is big.

This change is Reviewable

@ysv ysv requested a review from a team as a code owner March 2, 2026 14:07
@ysv ysv requested review from TxCorpi0x, masihyeganeh, metalarm10 and miladz68 and removed request for a team March 2, 2026 14:07

@ysv ysv left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@ysv made 1 comment.
Reviewable status: 0 of 2 files reviewed, 1 unresolved discussion (waiting on masihyeganeh, metalarm10, miladz68, and TxCorpi0x).


a discussion (no related file):
@metalarm10 @TxCorpi0x:

Intention:

  • all addresses which receive non-community PSE are excluded from community PSE distribution (we get them from mapping)
  • additionally legacy (Coreum) foundation address are excluded:
  core13xmyzhvl02xpz0pu8v9mqalsvpyy7wvs9q5f90
core14g6wpzdx8g9txvxxu3fl7fplal9y5ztx34ac5p
core1zn2ns3ls68jlsv5dgkuz0rxsxt5fhk7n9cfl23
core1p4gsfkmqm0uxua65phteqwnmu39fwjvtspfkcj
core1rddqzjzy4f5frxkhds3sux0m03encqtla3ayu9
core1qe7xz56v5sh4mr0vfq8qycnvjudgrslmjt0n3m
core17epxygqaytz5l63f0au04058kt4w72w6pkh0as

@TxCorpi0x TxCorpi0x left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@TxCorpi0x reviewed 4 files and all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on masihyeganeh, metalarm10, and miladz68).

@ysv ysv merged commit bde3abb into release-v6 Mar 3, 2026
19 of 22 checks passed
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.

3 participants