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

feat: add SemVerMixin #1034

Merged
merged 6 commits into from
Jan 21, 2025
Merged

feat: add SemVerMixin #1034

merged 6 commits into from
Jan 21, 2025

Conversation

0xClandestine
Copy link
Member

Optional extension of #1015.

Copy link
Collaborator

@ypatil12 ypatil12 left a comment

Choose a reason for hiding this comment

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

LGTM, approving. Just one clarifying question

src/contracts/interfaces/ISemVerMixin.sol Show resolved Hide resolved
Copy link
Collaborator

@ypatil12 ypatil12 left a comment

Choose a reason for hiding this comment

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

Let's do import ordering changes in a separate PR :)

@0xClandestine
Copy link
Member Author

Let's do import ordering changes in a separate PR :)

Removed, wondering if pods & token should also use SemVerMixin?

Copy link
Collaborator

@ypatil12 ypatil12 left a comment

Choose a reason for hiding this comment

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

LGTM, one minor nit but approving

script/utils/ExistingDeploymentParser.sol Outdated Show resolved Hide resolved
src/contracts/core/StrategyManagerStorage.sol Outdated Show resolved Hide resolved
src/contracts/interfaces/ISemVerMixin.sol Show resolved Hide resolved
@0xClandestine 0xClandestine merged commit 943d7ee into fix/signature-utils Jan 21, 2025
8 of 11 checks passed
@0xClandestine 0xClandestine deleted the feat/semver-mixin branch January 21, 2025 21:14
0xClandestine added a commit that referenced this pull request Jan 21, 2025
* feat: add `SemVerMixin`

* refactor: add `_majorVersion` helper for `SignatureUtils`

* test: passing

* test: passing

* refactor: rename EIP712_VERSION

* refactor: review changes
0xClandestine added a commit that referenced this pull request Jan 22, 2025
refactor: review changes

refactor: review changes

feat: remaining semver

feat: remaining semver

refactor: review changes

refactor: review changes

fix: ci

refactor: remove make cmd

fix: ci

fix: ci

feat: add `SemVerMixin` (#1034)

* feat: add `SemVerMixin`

* refactor: add `_majorVersion` helper for `SignatureUtils`

* test: passing

* test: passing

* refactor: rename EIP712_VERSION

* refactor: review changes

chore: forge fmt

refactor: natspec improvements

chore: forge fmt

fix: eip712 typehash

refactor: remove version overrides

refactor: remove make bump-version

refactor: immutable version

refactor: remove unused import

chore: `make bump-version VERSION=1`

refactor: `SignatureUtils` -> `SignatureUtilsMixin`

fix: bump-version.sh

chore: `make bump-version VERSION=1.0.3`

feat: add `make bump-version`

feat: add version fn

refactor: cleanup

fix: signature utils
0xClandestine added a commit that referenced this pull request Jan 29, 2025
refactor: review changes

refactor: review changes

feat: remaining semver

feat: remaining semver

refactor: review changes

refactor: review changes

fix: ci

refactor: remove make cmd

fix: ci

fix: ci

feat: add `SemVerMixin` (#1034)

* feat: add `SemVerMixin`

* refactor: add `_majorVersion` helper for `SignatureUtils`

* test: passing

* test: passing

* refactor: rename EIP712_VERSION

* refactor: review changes

chore: forge fmt

refactor: natspec improvements

chore: forge fmt

fix: eip712 typehash

refactor: remove version overrides

refactor: remove make bump-version

refactor: immutable version

refactor: remove unused import

chore: `make bump-version VERSION=1`

refactor: `SignatureUtils` -> `SignatureUtilsMixin`

fix: bump-version.sh

chore: `make bump-version VERSION=1.0.3`

feat: add `make bump-version`

feat: add version fn

refactor: cleanup

fix: signature utils
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