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

"Playful ping" - Update core ping plugin #108

Merged
merged 16 commits into from
Nov 28, 2024

Conversation

jeremiah-k
Copy link
Collaborator

@jeremiah-k jeremiah-k commented Nov 28, 2024

Update the ping plugin on the mesh side. It will match case and punctuation (!'s and ?'s up to 5).

!ping = !pong
Ping = Pong
Ping? = Pong?
PiNg!?! = PoNg!?!
PING!!!!? = PONG!!!!?

If more than 5 punctuation marks are used this is the result:
Ping!?!?!? = Pong...

jeremiah-k and others added 16 commits November 27, 2024 16:00
Previously, the configuration loader in `BasePlugin` only supported loading plugin configurations from the `plugins` section of the `config.yaml` file. This approach limited flexibility and made it incompatible with new configuration structures.

With this update, the configuration loader now supports additional sections, such as `community-plugins` and `custom-plugins`, while maintaining backward compatibility. Plugins can access their configurations regardless of where they are defined in the file.

- Added dynamic search across multiple levels: `plugins`, `community-plugins`, and `custom-plugins`.
- Ensured `plugins` is prioritized to avoid breaking existing setups.
- Plugins retain access to their full configuration through `self.config`.

How to access configuration in plugins:
- Use `self.config` to retrieve plugin-specific options.
- Example: `self.config.get('active', False)` checks if the plugin is active.
- All configuration options from the corresponding section (e.g., `plugins`, `community-plugins`) are available dynamically.
@jeremiah-k jeremiah-k merged commit 9f57b02 into geoffwhittington:main Nov 28, 2024
@jeremiah-k jeremiah-k deleted the update-plugins branch November 28, 2024 16:43
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