Skip to content
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.

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

Remove secrets usage #45

Merged
merged 1 commit into from
Mar 3, 2025
Merged

Conversation

justmobilize
Copy link
Contributor

Remove secrets usage

@justmobilize justmobilize marked this pull request as ready for review February 27, 2025 23:50
@justmobilize
Copy link
Contributor Author

@FoamyGuy this should be the last batch (11) of secret removals.

"Adafruit IO secrets are kept in settings.toml, please add them there!\n\n"
) from KeyError
aio_username = self._get_setting["ADAFRUIT_AIO_USERNAME"]
aio_key = self._get_setting["ADAFRUIT_AIO_KEY"]
Copy link
Contributor

Choose a reason for hiding this comment

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

Will the addition of the ADAFRUIT_ prefix to these keys require changing learn guide code or pages? Is that already coordinated if so?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So PortalBase will look for either, and I'm working with Tyeth on the learn guides

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

@FoamyGuy FoamyGuy left a comment

Choose a reason for hiding this comment

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

Thank you

@FoamyGuy FoamyGuy merged commit 7863e5b into adafruit:main Mar 3, 2025
1 check passed
@justmobilize justmobilize deleted the remove-secrets-usage branch March 3, 2025 14:48
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Mar 4, 2025
Updating https://github.com/adafruit/Adafruit_CircuitPython_DisplayIO_SSD1305 to 2.0.0 from 1.4.3:
  > Merge pull request adafruit/Adafruit_CircuitPython_DisplayIO_SSD1305#19 from FoamyGuy/use_ruff_remove_8x_comapt

Updating https://github.com/adafruit/Adafruit_CircuitPython_IL0398 to 2.0.0 from 1.2.4:
  > Merge pull request adafruit/Adafruit_CircuitPython_IL0398#17 from FoamyGuy/use_ruff_remove_8x_compat

Updating https://github.com/adafruit/Adafruit_CircuitPython_SSD1680 to 2.0.0 from 1.2.5:
  > Merge pull request adafruit/Adafruit_CircuitPython_SSD1680#29 from FoamyGuy/use_ruff_remove_8x_compat

Updating https://github.com/adafruit/Adafruit_CircuitPython_SSD1681 to 2.0.0 from 1.2.4:
  > Merge pull request adafruit/Adafruit_CircuitPython_SSD1681#15 from FoamyGuy/use_ruff_remove_8x_compat

Updating https://github.com/adafruit/Adafruit_CircuitPython_ST7735R to 2.0.0 from 1.6.4:
  > Merge pull request adafruit/Adafruit_CircuitPython_ST7735R#38 from FoamyGuy/use_ruff_remove_8x_compat

Updating https://github.com/adafruit/Adafruit_CircuitPython_ST7789 to 2.0.0 from 1.6.4:
  > Merge pull request adafruit/Adafruit_CircuitPython_ST7789#39 from FoamyGuy/use_ruff_remove_8x_compat

Updating https://github.com/adafruit/Adafruit_CircuitPython_UC8151D to 2.0.0 from 1.3.5:
  > Merge pull request adafruit/Adafruit_CircuitPython_UC8151D#14 from FoamyGuy/use_ruff_remove_8x_compat

Updating https://github.com/adafruit/Adafruit_CircuitPython_Wii_Classic to 1.0.9 from 1.0.8:
  > Merge pull request adafruit/Adafruit_CircuitPython_Wii_Classic#6 from FoamyGuy/use_ruff_remove_8x_compat

Updating https://github.com/adafruit/Adafruit_CircuitPython_FunHouse to 2.3.1 from 2.3.0:
  > Merge pull request adafruit/Adafruit_CircuitPython_FunHouse#45 from justmobilize/remove-secrets-usage

Updating https://github.com/adafruit/Adafruit_CircuitPython_Requests to 4.1.10 from 4.1.9:
  > Merge pull request adafruit/Adafruit_CircuitPython_Requests#208 from dhalbert/update-ap_info
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