Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sync ThingModel action failed #382

Closed
danielpeintner opened this issue Feb 10, 2022 · 6 comments · Fixed by #383 or #385
Closed

Sync ThingModel action failed #382

danielpeintner opened this issue Feb 10, 2022 · 6 comments · Fixed by #383 or #385

Comments

@danielpeintner
Copy link
Contributor

see https://github.com/w3c/wot-scripting-api/actions/runs/1821091154

it seems the previous "sync-td" job seems to cause the issue?

@relu91

@relu91
Copy link
Member

relu91 commented Feb 10, 2022

error: failed to push some refs to 'https://github.com/w3c/wot-scripting-api'
hint: Updates were rejected because the remote contains work that you do
hint: not have locally. This is usually caused by another repository pushing
hint: to the same ref. You may want to first integrate the remote changes
hint: (e.g., 'git pull ...') before pushing again.
hint: See the 'Note about fast-forwards' in 'git push --help' for details.

argh, I didn't take into account that the push of the previous job could cause some troubles. I think that the two jobs are run in parallel.... this makes things complicated... let me check

@danielpeintner
Copy link
Contributor Author

re-open since it did not work 😌

see https://github.com/w3c/wot-scripting-api/actions/runs/1826805143

@danielpeintner
Copy link
Contributor Author

@relu91 shall we split the 2 subtasks into 2 GitHub actions or do you want to give it another try?

@relu91
Copy link
Member

relu91 commented Feb 14, 2022

Splitting might work here, but now the action is falling for a different reason... I was thinking that the order of the steps were:

  • commit
  • pull
  • push

But actually, it first pulls which is falling cause we have un-committed changes 😢 . I'll to the docs again to see if I can change this behavior.

@relu91
Copy link
Member

relu91 commented Feb 14, 2022

Seems someone else's has the same use-case: EndBug/add-and-commit#359

@danielpeintner
Copy link
Contributor Author

today's scripting call

  • @relu91 proposes to create 2 GH actions. one depending on the other, in series.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants