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

Migrate Nina firmwares to plugin system #208

Merged
merged 7 commits into from Aug 9, 2023

Conversation

umbynos
Copy link
Contributor

@umbynos umbynos commented Aug 1, 2023

I've changed the generate.py python script to generate the required json file for the plugin_firmware_index.json file.
Along the way, I performed some optimizations. Further details in the commit messages.

Close #195

@umbynos umbynos added type: enhancement Proposed improvement topic: infrastructure Related to project infrastructure labels Aug 1, 2023
@umbynos umbynos self-assigned this Aug 1, 2023
@umbynos umbynos marked this pull request as draft August 1, 2023 15:30
…ds.json` file

This way we have to modify this file only to add support for new boards.
The `generate_boards_json()` function have been split: this way we can remove the old one when the time comes.
@umbynos umbynos changed the title Migrate Nina firmwares to plugin system Migrate Nina firmwares to plugin system and drop support for MKR 1000 and MKR Vidor 4000 Aug 2, 2023
@umbynos umbynos marked this pull request as ready for review August 2, 2023 10:43
@umbynos umbynos added the topic: documentation Related to documentation for the project label Aug 2, 2023
@codecov-commenter
Copy link

codecov-commenter commented Aug 2, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (bb28293) 6.63% compared to head (8eb340f) 6.63%.
Report is 5 commits behind head on main.

❗ Current head 8eb340f differs from pull request most recent head bf3cd6f. Consider uploading reports for the commit bf3cd6f to get more accurate results

Additional details and impacted files
@@          Coverage Diff          @@
##            main    #208   +/-   ##
=====================================
  Coverage   6.63%   6.63%           
=====================================
  Files         24      24           
  Lines       2154    2154           
=====================================
  Hits         143     143           
  Misses      1953    1953           
  Partials      58      58           
Flag Coverage Δ
unit 6.63% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
cli/arguments/arguments.go 0.00% <0.00%> (ø)
cli/certificates/flash.go 0.00% <0.00%> (ø)
cli/firmware/flash.go 0.00% <0.00%> (ø)
cli/firmware/getversion.go 0.00% <0.00%> (ø)
cli/firmware/list.go 0.00% <0.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@umbynos umbynos linked an issue Aug 2, 2023 that may be closed by this pull request
@umbynos umbynos changed the title Migrate Nina firmwares to plugin system and drop support for MKR 1000 and MKR Vidor 4000 Migrate Nina firmwares to plugin system Aug 3, 2023
@umbynos umbynos removed the topic: documentation Related to documentation for the project label Aug 3, 2023
@alessio-perugini alessio-perugini merged commit e241f76 into main Aug 9, 2023
7 checks passed
@alessio-perugini alessio-perugini deleted the migrate-nina-fw-plugin branch August 9, 2023 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: infrastructure Related to project infrastructure type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migrate Nina firmwares to the new plugin system
3 participants