ci: add an action to update submodules #174
Conversation
|
|
| autoupdate-branch = main | ||
| autoupdate-tag-glob = v[0-9]*.[0-9]*.[0-9]* | ||
| autoupdate-manifest = cbor/idf_component.yml | ||
| autoupdate-ver-regex = v([0-9]+).([0-9]+).([0-9]+) |
There was a problem hiding this comment.
Should we consider keeping some default values for branch, tag-glob and ver-regex fields? If we take most common syntax (e.g., version as vx.y.z) then we can reduce the additional fields specified for all external submodules.
There was a problem hiding this comment.
I kept these explicit in order to force the component maintainer to go and check the tag naming pattern the repository actually uses.
Alternatively I could make the action fail if not a single tag with the given naming pattern is found. In that case I'll also make the action run when .gitmodules file is changed, with a --dry-run argument.
| [submodule "esp_delta_ota/detools"] | ||
| path = esp_delta_ota/detools | ||
| url = https://github.com/eerimoq/detools.git | ||
| # not updating automatically, as this is not an upstream repo |
There was a problem hiding this comment.
We do use source from this repository in our component build. We should add this component in the auto update list.
There was a problem hiding this comment.
@mahavirj I can add it, but how is the version of the component related to the version of detools? It looks like they aren't related.
There was a problem hiding this comment.
Yes, they are not related. Our component follows its own versioning policy here. We can handle the update for this case on-demand basis.
|
@igrr Do we need any doc guidelines on the auto-update workflow for external submodules? |
|
LGTM! Let's wait for espressif/github-actions#40 |
igrr
force-pushed
the
feature/submodule_update
branch
from
6ea3b3d
to
5276490
Compare
igrr
force-pushed
the
feature/submodule_update
branch
from
5276490
to
8fba36e
Compare
This PR adds a new CI action to automatically update submodules, opening PRs for updates.
Example PRs can be seen in the fork:
Note, idf_component.yml wasn't updated in these PRs because:
in both cases, idf-extra-components maintainers will have to amend the PR to update idf_component.yml manually.
Uses the following Github action:
Description of config options in .gitmodules can be found the README.md of update_submodule_versions action.
TODO
Closes #29