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

chore: bump version #34

Merged
merged 7 commits into from
Jul 25, 2024
Merged

chore: bump version #34

merged 7 commits into from
Jul 25, 2024

Conversation

dudesahn
Copy link
Contributor

bump to use latest >= tokenized strategy and periphery. set 0.8.23 (last shanghai compiler version) as default for now

.gitmodules Outdated
branch = greater_than
Copy link
Collaborator

Choose a reason for hiding this comment

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

can you change this back to master

Copy link
Collaborator

Choose a reason for hiding this comment

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

and update the periphery submodule as well

@@ -2,7 +2,7 @@
src = 'src'
out = 'out'
libs = ['lib']
solc = "0.8.18"
solc = "0.8.23"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you just add a small comment to the Readme that says something like "Compiler defaults to 8.23 but it can be adjusted in the foundry toml"

@Schlagonia Schlagonia changed the base branch from master to periph July 25, 2024 20:52
@Schlagonia Schlagonia merged commit b2671ba into yearn:periph Jul 25, 2024
3 of 4 checks passed
Schlagonia added a commit that referenced this pull request Jul 25, 2024
* chore: remove old

* forge install: tokenized-strategy-periphery

6ce8d29b1e107a89754dd9f17337582734989b4d

* fix: remove version

* forge install: tokenized-strategy

v3.0.2-1

* fix: module versions

* chore: bump version (#34)

* chore: bump version

bump to 0.8.23

* chore: update submodules

* fix: greater than 18

* fix: redo requirements

* feat: one more bump

* fix: use console2

---------

Co-authored-by: Schlag <[email protected]>

* feat: comment

---------

Co-authored-by: dudesahn <[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