Skip to content

Conversation

evaherrada
Copy link
Collaborator

No description provided.

@evaherrada evaherrada requested a review from kattni April 14, 2020 20:42
@evaherrada
Copy link
Collaborator Author

This one definitely needs to be looked through as there was definitely some wackiness with me forgetting to make a PR a few weeks ago, but I've looked through it and it looks good to me

@kattni
Copy link
Contributor

kattni commented Apr 14, 2020

@dherrada This PR is only showing one file changed. Please look into what's going on.

@evaherrada
Copy link
Collaborator Author

@kattni I think what happened is that I pushed the first commit a while back, but never made a PR. When you went and updated the black version, it did the exact same thing I did in the second commit. It seems like the only thing that needs to be changed here is the removal of ==1.92 in build.yml so I'm happy to either just edit master or make a pr from a different branch.

@kattni kattni merged commit cbc8c46 into master Apr 15, 2020
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Apr 24, 2020
Updating https://github.com/adafruit/Adafruit_CircuitPython_AM2320 to 1.2.2 from 1.2.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_AM2320#16 from jerryneedell/jerryn_revert_pr_9

Updating https://github.com/adafruit/Adafruit_CircuitPython_LIS2MDL to 2.1.3 from 2.1.2:
  > Merge pull request adafruit/Adafruit_CircuitPython_LIS2MDL#5 from FoamyGuy/master

Updating https://github.com/adafruit/Adafruit_CircuitPython_LSM6DS to 2.1.3 from 2.1.2:
  > Merge pull request adafruit/Adafruit_CircuitPython_LSM6DS#14 from adafruit/rads_fix
  > Merge pull request adafruit/Adafruit_CircuitPython_LSM6DS#13 from adafruit/pylint-update

Updating https://github.com/adafruit/Adafruit_CircuitPython_BLE to 6.1.3 from 6.1.2:
  > Merge pull request adafruit/Adafruit_CircuitPython_BLE#80 from dhalbert/initial-value-none
@evaherrada evaherrada deleted the pylint-update branch August 6, 2020 22:15
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