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

Update arduino-ci-script subtree to 27f2438 #43

Merged
merged 1 commit into from
May 16, 2017
Merged

Update arduino-ci-script subtree to 27f2438 #43

merged 1 commit into from
May 16, 2017

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented May 15, 2017

Build was successful!
https://travis-ci.org/per1234/MegaCore/builds/232435609

The commits included in this update should solve the problem of report
push conflicts when two jobs finish at about the same time.

per1234/arduino-ci-script@27f2438

The commits included in this update should solve the problem of report
push conflicts when two jobs finish at about the same time.

per1234/arduino-ci-script@27f2438
@MCUdude
Copy link
Owner

MCUdude commented May 16, 2017

Great! Looking forward to finally merge this branch into the master.

@MCUdude MCUdude merged commit 0e31f8f into MCUdude:avr-100-pin May 16, 2017
@MCUdude
Copy link
Owner

MCUdude commented May 16, 2017

Seems like the issue isn't 100% resolved just yet.. seems like it's only two jobs that's failed though. https://travis-ci.org/MCUdude/MegaCore/builds/232706002

@per1234
Copy link
Contributor Author

per1234 commented May 16, 2017

That's strange that my builds were successful but yours failed.

I believe I have now fixed the script's error handling behavior so that now it will retry the push multiple times (which should almost guarantee success).

If for some reason it still can't push the report it will give up, complete the rest of the build lifecycle steps, and the build will still be considered success, the report for that job will just be missing.

I will submit a PR once all tests have finished.

@per1234 per1234 deleted the update_arduino-ci-script branch May 17, 2017 00:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants