Skip to content

Fix pylint errors #29

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Nov 8, 2022
Merged

Fix pylint errors #29

merged 1 commit into from
Nov 8, 2022

Conversation

tekktrik
Copy link
Member

@tekktrik tekktrik commented Nov 8, 2022

Fixes the following pylint errors:

************* Module adafruit_fancyled.adafruit_fancyled
Error: adafruit_fancyled/adafruit_fancyled.py:183:23: C0103: Argument name "h" doesn't conform to '(([a-z][a-z0-9_]{2,30})|(_[a-z0-9_]*))$' pattern (invalid-name)
Error: adafruit_fancyled/adafruit_fancyled.py:183:26: C0103: Argument name "s" doesn't conform to '(([a-z][a-z0-9_]{2,30})|(_[a-z0-9_]*))$' pattern (invalid-name)
Error: adafruit_fancyled/adafruit_fancyled.py:183:33: C0103: Argument name "v" doesn't conform to '(([a-z][a-z0-9_]{2,30})|(_[a-z0-9_]*))$' pattern (invalid-name)

@tekktrik tekktrik requested review from kattni and evaherrada November 8, 2022 04:46
@evaherrada evaherrada merged commit 13a6392 into adafruit:main Nov 8, 2022
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Nov 15, 2022
@adafruit-adabot
Automated update by Adabot (adafruit/adabot@a295390)
8279211 
Updating https://github.com/adafruit/Adafruit_CircuitPython_ESP_ATcontrol to 0.9.1 from 0.9.0:
  > Merge pull request adafruit/Adafruit_CircuitPython_ESP_ATcontrol#67 from tekktrik/dev/fix-errors
  > Update .pylintrc for v2.15.5
  > Fix release CI files
  > Update pylint to 2.15.5
  > Updated pylint version to 2.13.0
  > Switching to composite actions

Updating https://github.com/adafruit/Adafruit_CircuitPython_PN532 to 2.3.14 from 2.3.13:
  > Update .pylintrc for v2.15.5
  > Fix release CI files
  > Update pylint to 2.15.5
  > Updated pylint version to 2.13.0
  > Switching to composite actions

Updating https://github.com/adafruit/Adafruit_CircuitPython_FancyLED to 1.4.14 from 1.4.13:
  > Merge pull request adafruit/Adafruit_CircuitPython_FancyLED#29 from tekktrik/dev/fix-pylint-errors
  > Update .pylintrc for v2.15.5
  > Fix release CI files
  > Update pylint to 2.15.5
  > Updated pylint version to 2.13.0
  > Switching to composite actions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@evaherrada evaherrada evaherrada approved these changes

@kattni kattni Awaiting requested review from kattni

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tekktrik @evaherrada