Skip to content

Conversation

FoamyGuy
Copy link
Contributor

The circup install command in the readme was excluding the adafruit_ prefix so that command was not able to complete successfully.

This change updates the command to:

circup install adafruit_ticks

I tested the updated command succesfully with a FunHouse on 7.0.0 alpha5 (though device an version shouldn't make a different I don't think).

@FoamyGuy FoamyGuy requested a review from a team November 17, 2021 16:44
Copy link
Contributor

@dhalbert dhalbert left a comment

Choose a reason for hiding this comment

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

thanks!

@dhalbert dhalbert merged commit 676731a into adafruit:main Nov 17, 2021
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Nov 24, 2021
Updating https://github.com/adafruit/Adafruit_CircuitPython_LC709203F to 2.2.0 from 2.1.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_LC709203F#17 from kattni/add-400-pack-size
  > update rtd py version

Updating https://github.com/adafruit/Adafruit_CircuitPython_PCF8563 to 1.0.0 from 0.9.0:
  > Moved PR template
  > Merge pull request adafruit/Adafruit_CircuitPython_PCF8563#3 from tekktrik/hotfix/fix-errors
  > Updated readthedocs file
  > Updated readthedocs requirements path
  > add docs link to readme
  > Added pylint disable for f-strings in tests directory
  > Globally disabled consider-using-f-string pylint check

Updating https://github.com/adafruit/Adafruit_CircuitPython_BLE to 8.1.2 from 8.1.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_BLE#144 from FoamyGuy/rtd_version
  > Merge pull request adafruit/Adafruit_CircuitPython_BLE#142 from adafruit/patch-fix

Updating https://github.com/adafruit/Adafruit_CircuitPython_Ticks to 1.0.0 from 0.9.2:
  > Merge pull request adafruit/Adafruit_CircuitPython_Ticks#3 from FoamyGuy/install_command
  > Merge pull request adafruit/Adafruit_CircuitPython_Ticks#2 from adafruit/patch
  > Updated readthedocs file
  > Updated readthedocs requirements link
  > Added pylint disable for f-strings in tests directory
  > Globally disabled consider-using-f-string pylint check
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.

3 participants