transmissions11 / solmate

Modern, opinionated, and gas optimized building blocks for smart contract development.
GNU Affero General Public License v3.0
3.93k stars 645 forks source link

Remove `totalSupply`. Add checked balance arithmetic. #397

Closed jtriley-eth closed 11 months ago

jtriley-eth commented 11 months ago

Description

Removes totalSupply in accordance with EIP-6909 update.

Adds checked arithmetic on balance increases, as they previously relied on a checked addition to totalSupply

Note: npm run lint corrects src/test/CREATE3.t.sol. Disregarding for simplicity.

Checklist

Ensure you completed all of the steps below before submitting your pull request:

jtriley-eth commented 11 months ago

something's up with the ci, forge snapshot --check is failing, even on solmate main

gonna halt this for now and open a pr for a fix