Nick Sarten

Ah sorry, I just pished the patch I created, but there's more changes in our local copy that haven't been upstreamed, I'll add them to this PR.

Ok so the further changes make basic_tree::parse_text a cont method, and implement locking such that all const methods are thread safe, which is where this bug originally comes from. So...

I have pushed all of the changes. As I have already said, you're free to choose not to take them if you don't think this functionality is valuable. It certainly...

Ah, it appears I didn't fully understand the intention of the code. Ignoring the warning may be a better solution in this case if the behaviour is intended.

That sounds right. Possibly you could automatically enable half-duplex mode when the MOSI and MISO pins are the same and keep the same API?

Note that I'm referring to multidirectional half-duplex transmission, not write-only or receive-only unidirectional half-duplex. I'm not sure if the dummy byte approach would allow the user to send and receive...

Looking into it further it seems that all the half duplex hardware mode does is tie the MOSI and MISO pins together internally, saving an external pin. For a send...

Wow now I really feel like an idiot, I've literally been writing my own drivers for the LSM303 and LIS3MDL for the last couple of months. I can't believe I...

Uh to add a little context, this is now a slightly bigger issue than just forks, as all dependabot PRs [now run with read-only permissions](https://github.blog/changelog/2021-02-19-github-actions-workflows-triggered-by-dependabot-prs-will-run-with-read-only-permissions/) due to security issues. We've...

This is the workaround I've put in place for now to run clippy manually when a workflow doesn't have write permissions: ``` - name: Check workflow permissions id: check_permissions uses:...