-
Notifications
You must be signed in to change notification settings - Fork 551
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
docs: change docs to use "apps" instead of "plugins" #3651
Conversation
Visit the preview URL for this PR (updated for commit 1302318): https://igntservices-docs--pr3651-docs-ignite-apps-tf68c8g6.web.app (expires Tue, 03 Oct 2023 07:46:21 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 95379efd94dd497aaa37c2d0354e6e2cafca5ec5 |
If this PR is approved it would require a small refactor to the "CLI plugins API" to change the user facing API from The docs use the concept of Ignite Apps except in some parts that describe the plugin system, but it is explained that under the hood Ignite Apps are implemented using a plugin system based on Hashicorp's plugins. I think is the fastest approach, but we could also decide for a bigger refactor to change the code to also use the concept of app instead of plugin. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
Resolves #3647