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

[Rust] Generation: Avoid warnings #995

Merged

Conversation

YuhanunCitgez
Copy link
Contributor

This PR intends to avoid warnings that get generated because of some clippy flags.

We glob re-export the statics generated within the codecs, which causes issues due to duplicate naming.

In an ideal world, we would stop exporting the statics (or naming them uniquely), but for now I'd like to just avoid the warnings.

Additionally; get_bytes is unused and unusable outside of the generated code itself, rendering it useless for any real-world project.

@YuhanunCitgez
Copy link
Contributor Author

No clue why this failed 😓 Only Rust generation has changed. Is this an already failing test? @vyazelenko

@vyazelenko
Copy link
Contributor

@YuhanunCitgez Try merging 0053ff3 commit into your branch.

@YuhanunCitgez
Copy link
Contributor Author

@vyazelenko Thanks, pushed.

Let me know if there's anything I need to do to get this into stable, tried to follow CONTRIBUTING as far as I could hahaha

@YuhanunCitgez
Copy link
Contributor Author

@vyazelenko shall we merge?

@YuhanunCitgez
Copy link
Contributor Author

Not sure how to push ahead here.

Pinging @vyazelenko and @mjpt777 :)

@vyazelenko vyazelenko merged commit 2a2d308 into real-logic:master Jun 18, 2024
34 checks passed
@YuhanunCitgez YuhanunCitgez deleted the rust-generation-warnings-fix branch July 25, 2024 16:03
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