search

eclipse-platform / eclipse.platform.text

8 stars 45 forks source link

Add updateRelease.yml #156

Closed akurtakov closed 1 year ago

akurtakov commented 1 year ago

@laeubi Any idea what went wrong in https://github.com/akurtakov/eclipse.platform.text/actions/runs/4352345143/jobs/7605013788 ?

akurtakov commented 1 year ago

I get what is wrong - it tries git add **/pom.xml but everything is pomless so updateRelease.yml needs a change. I'll push the change as the fix to the action has to go in the action repo itself.

laeubi commented 1 year ago

@akurtakov the problem is that I don't have enough git-power to get something that includes ANY pom.xml (including the root!) so i used two path specs but it seems git for some reason don't like that .... its a bit of a mystery to me ;-)

https://github.com/eclipse-platform/eclipse.platform.releng.aggregator/blob/6c0c78077ab8d8ef326b55f0a9191f3dc93c93e9/.github/workflows/updateRelease.yml#L45-L48

on the other hand it does not complains about **/*.MF what also has no changes...

laeubi commented 1 year ago

I'll push the change as the fix to the action has to go in the action repo itself.

You can also use a forked "releng" repo for experiments with the "included" workflow :-)

akurtakov commented 1 year ago

It is probably bailing on first issue and not even trying next.

laeubi commented 1 year ago

Yes probabbly but we already have had sucessfull runs, so its a bit confusing (for me), so if you can suggest a more gracefull pattern that would be great!