SYS-331: Update `_REASONS` map with new signature and gas reasons #51

muni-corn · 2024-08-05T22:57:47Z

github-actions · 2024-08-05T22:58:22Z

PR Reviewer Guide 🔍

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Key issues to review Redundant Code The new entries 'Transaction is not signed' and 'Transaction signature is invalid' might be redundant or overlapping with the previous 'Invalid signature' entry. Consider consolidating these to avoid potential confusion and maintain code simplicity.

github-actions bot added the Review effort [1-5]: 2 label Aug 5, 2024

chrypnotoad requested a review from a team August 6, 2024 15:37

mssabr01 previously approved these changes Aug 6, 2024

View reviewed changes

chrypnotoad dismissed mssabr01’s stale review via 0b548d4 August 6, 2024 19:33

chrypnotoad requested a review from mssabr01 August 7, 2024 15:28

mssabr01 previously approved these changes Aug 7, 2024

View reviewed changes

muni-corn and others added 2 commits August 20, 2024 21:09

fix(utils): update _REASONS map with new signature and gas reasons

3ed4eb7

fix patch failing PR check, add check-patches npm script

d390342

mhanson-github dismissed mssabr01’s stale review via d390342 August 21, 2024 01:12

mhanson-github force-pushed the sys-331 branch from 787878c to d390342 Compare August 21, 2024 01:12

mhanson-github merged commit d390342 into dev Aug 21, 2024
6 of 7 checks passed

mhanson-github deleted the sys-331 branch August 30, 2024 20:16