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

Plugins mechanism #910

Merged
merged 23 commits into from
Oct 17, 2023
Merged

Plugins mechanism #910

merged 23 commits into from
Oct 17, 2023

Conversation

rangoo94
Copy link
Member

@rangoo94 rangoo94 commented Oct 12, 2023

Changes

  • Added a base for the plugins system
    • It's not integrated yet but will be in further PRs
    • To see usage examples, take a look at the unit tests or the README file
  • Don't worry that much about the code size
    • Most of the PR consists of unit tests, documentation and configuration (90-100% of code coverage)

How to test it

  • Validate if the documentation is clear
  • Validate unit tests examples

Checklist

  • tested locally
  • added new dependencies
  • updated the docs
  • added a test

@vercel
Copy link

vercel bot commented Oct 12, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
testkube-dashboard ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 17, 2023 9:39am

@rangoo94 rangoo94 force-pushed the dawid/feat/plugins-mechanism branch from 0c16816 to 2ff8a0b Compare October 17, 2023 09:35
@rangoo94 rangoo94 merged commit cbda693 into develop Oct 17, 2023
@rangoo94 rangoo94 deleted the dawid/feat/plugins-mechanism branch October 17, 2023 11:07
haneabogdan pushed a commit that referenced this pull request Oct 24, 2023
* feat: prepare initial plugins mechanism
* fixup silent console.warns
* feat: synchronize plugin scope data on plugin level (instead of root)
* chore: rename LimitedPluginScope to LocalPluginScope
* fix: ts-check for plugins package
* feat: replace LocalPluginScope/PluginRootScope with universal PluginScope
* chore: delete unused provider
* feat: support nested plugin systems (outer root scope / outer dependencies)
* chore: add unit tests to avoid warnings for missing outer data/slots
* fix: ensure console.warn mock stability
* feat: support clearing data produced within the PluginScope
* chore: add a unit test for clearing PluginScope
* fix: children plugin scope should not be detached from the parent
* chore: replace Array with Set
* feat: improve code completion for external slots/data
* feat: allow multiple slots/data definition with same type within same call
* feat: add option to provide conditional providers
* feat: add documentation for the Plugins system
* feat: add documentation for Zustand store in the Plugins system
* feat: add option to use scope data in providers easily
@github-actions github-actions bot mentioned this pull request Nov 20, 2023
4 tasks
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