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

Add documentation on the new plugin system #178

Merged

Conversation

MatteoPologruto
Copy link
Contributor

The documentation on the new plugin system has been added.

@MatteoPologruto MatteoPologruto added type: enhancement Proposed improvement topic: documentation Related to documentation for the project labels Jul 19, 2023
@MatteoPologruto MatteoPologruto self-assigned this Jul 19, 2023
Copy link
Contributor

@per1234 per1234 left a comment

Choose a reason for hiding this comment

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

Please add plugins.md to the nav key of the mkdocs.yml configuration file:

https://github.com/arduino/arduino-fwuploader/blob/main/mkdocs.yml

This will cause the page to be listed in the navigation panel of the website.

Reference:
https://www.mkdocs.org/user-guide/configuration/#nav

docs/plugins.md Show resolved Hide resolved
[interface](https://github.com/arduino/fwuploader-plugin-helper/blob/main/plugin.go#L28) contained in the
[fwuploader-plugin-helper](https://github.com/arduino/fwuploader-plugin-helper) repository.

The fwuploader is still responsible for downloading the files required for the plugin to work:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
The fwuploader is still responsible for downloading the files required for the plugin to work:
Arduino Firmware Uploader is still responsible for downloading the files required for the plugin to work:

docs/plugins.md Show resolved Hide resolved
docs/plugins.md Show resolved Hide resolved
docs/plugins.md Show resolved Hide resolved
docs/plugins.md Show resolved Hide resolved
@alessio-perugini alessio-perugini changed the base branch from main to add-new-plugin-docs July 20, 2023 10:20
@alessio-perugini
Copy link
Contributor

@per1234 since @MatteoPologruto is OOO and he created a fork I'm merging this into a new branch so I can apply your suggestions in a new commit 🙏

@alessio-perugini alessio-perugini merged commit ac044a9 into arduino:add-new-plugin-docs Jul 20, 2023
6 checks passed
alessio-perugini added a commit that referenced this pull request Jul 20, 2023
* Add documentation on the new plugin system (#178)

* Apply Code Review suggestions

---------

Co-authored-by: MatteoPologruto <109663225+MatteoPologruto@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: documentation Related to documentation for the project type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants