Closed lemeurherve closed 4 months ago
As noted in:
The updatecli manifests with upper cases looks weird though Maybe due to the old "pullrequest" syntax used in the manifest? https://github.com/jenkins-infra/docker-confluence-data/blob/d8f5186e941cd6541ff3aa4a3b25c4bd064ac2ab/updatecli/updatecli.d/nginx.yaml#L56-L65
The updatecli manifests with upper cases looks weird though
Maybe due to the old "pullrequest" syntax used in the manifest? https://github.com/jenkins-infra/docker-confluence-data/blob/d8f5186e941cd6541ff3aa4a3b25c4bd064ac2ab/updatecli/updatecli.d/nginx.yaml#L56-L65
The updatecli manifest should be updated to replace the deprecated pullrequests section by an actions one: https://www.updatecli.io/docs/plugins/actions/github/
pullrequests
actions
This manifest should also update the tests file so it can test the correct version: https://github.com/jenkins-infra/docker-confluence-data/blob/d8f5186e941cd6541ff3aa4a3b25c4bd064ac2ab/cst.yml#L3-L7
Fixed by #63 and #65
The Nginx version was upgraded with success after these PRs, in #64
As noted in:
The updatecli manifest should be updated to replace the deprecated
pullrequests
section by anactions
one: https://www.updatecli.io/docs/plugins/actions/github/This manifest should also update the tests file so it can test the correct version: https://github.com/jenkins-infra/docker-confluence-data/blob/d8f5186e941cd6541ff3aa4a3b25c4bd064ac2ab/cst.yml#L3-L7