idf-extra-components icon indicating copy to clipboard operation
idf-extra-components copied to clipboard

Consider setting up dependabot to update submodule dependencies (IEC-2)

Open igrr opened this issue 3 years ago • 4 comments

Dependabot has basic support for tracking and upgrading dependencies expressed using git submodules (https://docs.github.com/en/code-security/dependabot/dependabot-version-updates/configuration-options-for-the-dependabot.yml-file#package-ecosystem). Consider setting it up for this repository to get notified whenever a dependency can be upgraded.

Note that we might need to wait for or contribute to https://github.com/dependabot/dependabot-core/issues/1639 first, as currently dependabot will try to upgrade to the latest commit, not to the latest tag.

igrr avatar Mar 19 '22 04:03 igrr

Done.

I don't have any experience with dependabot, so I'll keep this issue open for future evaluation/discussion

tore-espressif avatar Mar 25 '22 18:03 tore-espressif

Thanks @tore-espressif!

The PRs opened by dependabot have two problems now:

  1. The PR updates the submodule to the latest commit, not to the latest tag (issue https://github.com/dependabot/dependabot-core/issues/1639 noted above).
  2. Looks like the "publish test results" job is failing.

Still even in this form they are useful as a hint/reminder to us that some dependency might be outdated.

Edit: plus we need to remember to bump the version in idf_component.yml when the dependency is upgraded. So looks like we'll be taking over dependabot PRs anyway.

igrr avatar Mar 25 '22 19:03 igrr

Given the above limitations, I'm thinking of adding a custom CI workflow instead of dependabot... This workflow could also update our idf_component.yml files and make the PRs mergeable.

igrr avatar Mar 28 '22 14:03 igrr

This might be worth revisiting, especially since https://github.com/espressif/idf-extra-components/pull/146 adds two libraries which are known to sometimes have CVEs reported for them. It would be good to be able to get the new releases of these libraries published quickly.

igrr avatar Feb 20 '23 19:02 igrr