-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[V3] Plugins implemenations #3570
Conversation
Caution Review failedThe pull request is closed. WalkthroughThe changes involve a significant transition from a plugin-based architecture to a service-oriented model within the application. This includes the introduction of new service functionalities, updates to existing commands and structures, and modifications to documentation. Key enhancements include the implementation of service lifecycle management, improved HTTP handling, and the reorganization of service definitions and interfaces to support the new architecture. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Service
participant Application
User->>Application: Start Application
Application->>Service: OnStartup()
Service-->>Application: Initialization Complete
User->>Service: Request Service Functionality
Service-->>User: Provide Response
Application->>Service: OnShutdown()
Service-->>Application: Cleanup Complete
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
9e31127
to
cc2f50c
Compare
remove log trim path prefix wails/services
bb3e2e8
to
6c167f6
Compare
Quality Gate failedFailed conditions See analysis details on SonarCloud Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
@atterpac - any chance we could send those errors via the new error handlers? 🙏 |
Do we want to consider a plugin error as fatal? |
… generation. Update README
# Conflicts: # v3/pkg/application/application.go # v3/pkg/application/plugins.go
Quality Gate failedFailed conditions See analysis details on SonarCloud Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
Adds plugins as a service as discussed in #3496
See discussion for more information until PR is finalized
Summary by CodeRabbit
New Features
Bug Fixes
Documentation
Style