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: update solana crates #74

Merged
merged 8 commits into from
Aug 8, 2024
Merged

Conversation

rwwwx
Copy link
Contributor

@rwwwx rwwwx commented Aug 8, 2024

Nothing special, just update Solana crates versions. This update is necessary for reduce dependency hell in Utility-Chain repo.

Since we updated solana crates versions to 1.18.*, the definition of the GeyserPlugin trait has changed. The _is_reload argument was added to the on_load function. I added this argument and logged it. Also I fixed some fmt and warnings from Сargo.

Also updated crates versions:
plerkle from 1.6.0 to 1.7.0
plerkle_serialization from 1.8.0 to 1.9.0
plerkle_messenger from 1.8.0 to 1.8.1

TODO: We have a lot of warnings and errors on ci from clippy, I didn't fix it because I'm not sure if it's my area of ​​responsibility, but if I'm wrong please let me know and I'll fix it.

Additional info about this on_load flag:
solana-labs/solana#33674
solana-labs/solana#24871
solana-labs/solana#30352

Copy link

@StanChe StanChe left a comment

Choose a reason for hiding this comment

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

Great, thanks. Can you also bump up the crate version so we could publish it later?

@rwwwx
Copy link
Contributor Author

rwwwx commented Aug 8, 2024

@danenbm Plz take a look on this PR, @RequescoS said that your are working currently on the same thing

blockiosaurus
blockiosaurus previously approved these changes Aug 8, 2024
plerkle_messenger/Cargo.toml Outdated Show resolved Hide resolved
@rwwwx rwwwx requested review from StanChe and blockiosaurus August 8, 2024 17:14
.github/workflows/release.yml Outdated Show resolved Hide resolved
@danenbm danenbm merged commit dd8744b into metaplex-foundation:main Aug 8, 2024
1 check passed
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.

5 participants