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
platformio: 5.0.4 -> 5.1.1 #114014
platformio: 5.0.4 -> 5.1.1 #114014
Conversation
4bb296e
to
a74c244
Compare
@ofborg eval |
I have updated and rebased this pull request for PlatformIO's latest release 5.1.1. This introduced a new dependency, ajsonrpc, for which I created another PR #116727. However, to make this changes work, the starlette package update needs to be merged first, #112701. I have successfully tested this package with those two changes applied locally. |
After all necessary dependencies were merged, I rebased this PR once again and now it should be ready for a final review. |
FWIW I have also tested this locally and it is working as intended. |
@SuperSandro2000: Would you please take another look? Thanks! |
Result of |
The PR looks good to me, doesn't seem to be a major change anyway. |
Motivation for this change
Update platformio to its latest release. However, to get this changes working, the following pull requests needed to be merged first:
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)