Skip to content

Conversation

@hugodahl
Copy link
Contributor

Fixing action issues from Infrastructure issues list for GitHub action failures

@tannewt tannewt requested a review from evaherrada March 30, 2021 17:50
@evaherrada
Copy link
Collaborator

@tannewt If I remember correctly this is an issue with the python version in build.yml. Do you happen to remember which version is the right one to use (I'll make sure to make a note of it somewhere so I don't forget)

@dhalbert
Copy link
Contributor

dhalbert commented Apr 6, 2021

@tannewt If I remember correctly this is an issue with the python version in build.yml. Do you happen to remember which version is the right one to use (I'll make sure to make a note of it somewhere so I don't forget)

You need at least 3.7

@evaherrada
Copy link
Collaborator

Cool. That's what I suspected it was. Thanks!

@evaherrada
Copy link
Collaborator

Not really sure about the current failure

@dhalbert
Copy link
Contributor

dhalbert commented Apr 6, 2021

Not really sure about the current failure

This is bleak trying to use bluetoothctl without checking for it not being there. I'll check on this.

@dhalbert
Copy link
Contributor

dhalbert commented Apr 6, 2021

Uncomment and change this line to this new version in docs/conf.py:

autodoc_mock_imports = ["bleak", "digitalio", "busio"]

@evaherrada
Copy link
Collaborator

Will do. Thanks!

@tannewt tannewt merged commit 69fdf47 into adafruit:main Apr 8, 2021
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Apr 10, 2021
Updating https://github.com/adafruit/Adafruit_CircuitPython_BLE_LYWSD03MMC to 1.0.2 from 1.0.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_BLE_LYWSD03MMC#2 from adafruit/tannewt-patch-1
  > Merge pull request adafruit/Adafruit_CircuitPython_BLE_LYWSD03MMC#1 from hugodahl/feature/Fix-GitHub-action-issues
  > "Increase duplicate code check threshold "
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.

4 participants