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

Upgrade to sdk 38.0 #106

Merged
merged 2 commits into from
Sep 13, 2023
Merged

Upgrade to sdk 38.0 #106

merged 2 commits into from
Sep 13, 2023

Conversation

Nereuxofficial
Copy link
Contributor

It would probably good to make a dev branch for this as it has not been extensively tested.

I used this awesome crate to play a bit with the flipper zero and it's really cool that this crate enables that without external bindings! Though the SDK Version was incompatible so i updated it. I also updated the documentation for some examples as I went through them

@Nereuxofficial
Copy link
Contributor Author

Also a small observation: I saw cargo-runner.py use run-fap from the tools. Could this be used as a runner for the flipperzero-template to avoid uploading and executing manually, improving developer experience? If so, I'd love to implement that!

crates/sys/src/bindings.rs Outdated Show resolved Hide resolved
crates/flipperzero/examples/threads.rs Show resolved Hide resolved
@dcoles
Copy link
Collaborator

dcoles commented Sep 8, 2023

Also a small observation: I saw cargo-runner.py use run-fap from the tools. Could this be used as a runner for the flipperzero-template to avoid uploading and executing manually, improving developer experience? If so, I'd love to implement that!

Absolutely! That's a great idea!

We've considering making the tools a requirement for most development. The cargo-runner.py script was an attempt to have things work automatically; but it never quite worked 100% (especially if you wanted to support multiple OSs). So my recommendation would be to just set the runner to run-fap directly.

@dcoles dcoles changed the title Upgrade to sdk 36.1 Upgrade to sdk 38.0 Sep 13, 2023
Copy link
Collaborator

@dcoles dcoles left a comment

Choose a reason for hiding this comment

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

I updated the bindings to 38.0 since the current release candidate for 0.91 is now up to that value.

@dcoles dcoles merged commit efe93e6 into flipperzero-rs:main Sep 13, 2023
7 of 9 checks passed
@dcoles
Copy link
Collaborator

dcoles commented Sep 13, 2023

Thanks for the PR!

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