Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@danlapid
Copy link
Contributor

This is my suggestion for a fix as suggested in #7338

@github-actions github-actions bot added the rust label Sep 27, 2022
@CasperN
Copy link
Collaborator

CasperN commented Sep 27, 2022

Thanks! Generally LGTM with some nits

@CasperN
Copy link
Collaborator

CasperN commented Oct 3, 2022

Hello, can you pull again? #7518 had a lot of Rust changes

@danlapid
Copy link
Contributor Author

@CasperN Pulled and merged, still working and passing on my pc.

@danlapid
Copy link
Contributor Author

@CasperN Build and CI seems to be passing, anything else we need to get this merged?

@CasperN
Copy link
Collaborator

CasperN commented Oct 19, 2022

Good to merge. Are you able to push the button, or should I?

@danlapid
Copy link
Contributor Author

Good to merge. Are you able to push the button, or should I?

I don't have access to that haha.
Go ahead :)

@CasperN CasperN merged commit 5792623 into google:master Oct 19, 2022
bulentv pushed a commit to zdrealityhub/flatbuffers that referenced this pull request Oct 21, 2022
* Fix nightly no_std

* Fix nightly no_std
jochenparm pushed a commit to jochenparm/flatbuffers that referenced this pull request Oct 29, 2024
* Fix nightly no_std

* Fix nightly no_std
jochenparm pushed a commit to jochenparm/flatbuffers that referenced this pull request Oct 29, 2024
* Fix nightly no_std

* Fix nightly no_std
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants