search

ethereum / solidity-website

26 stars 13 forks source link

Solidity v0.8.24 release announcement #77

Closed mehtavishwa30 closed 9 months ago

mehtavishwa30 commented 9 months ago

@cameel I made a couple of larger changes in the sections to address open comments. Please review and lmk what you think.

cameel commented 9 months ago

I made a couple of larger changes in the sections to address open comments. Please review and lmk what you think.

@mehtavishwa30 It's looking good. I restructured it a bit and added detail, which ended up as a bigger update. In particular I added some compiler dev perspective, since it was a bit too focused on the EIPs themselves. But it was already a good base and now is basically a combination of the two, which I think ended up nicely. Please take a look and feel free to do any final corrections. Then you can merge it unless anyone else has some additions.

@r0qs Could you take another look? The text changed a lot and some of your comments no longer apply. I applied those that still made sense in the current version.

cameel commented 9 months ago

BTW, please remember to squash before merging.

cameel commented 9 months ago

How does merging work here - I assume it auto deploys, which means we have to merge it at roughly at the same time as the transient storage post?

Yes, merge triggers website build and deployment.

nikola-matic commented 9 months ago

@mehtavishwa30 if you see this tomorrow and wanna merge before the rest of us get online, it would be best to remove the link (and mention) of the transient store blog post, so that we can merge (and thus publish) the release post, and then subsequently alter in publish the transient storage blog post as well.

cameel commented 9 months ago

Transient storage post is actually mergeable. We just finished working in it and it's approved.