fix: correct updatecli golang manifest to update go.mod #294
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This should lead to a successful update of #273 once merged.
As explained in #273 (comment), there is an issue with the current
go.mod
update process:This PR uses a shell resource as a "source", to update the go version of the go mod in a temp dir, and prints the content of the tidied resuting go.mod file.
Please note that this PR also enables the
updatecli
workflow for all PR/branches:updatecli diff
command is always executed (to ensure that it's not broken when there are configuration changes)updatecli apply
command is only executed on themain
branch of course=> The goal is to validate that the new shell script resource works as expected (for instance that it successfully installs golang when runing withing its image)