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

[10.10.1] fast-fail false; do not abort other builds #4153

Merged

Conversation

nerdCopter
Copy link
Member

@nerdCopter nerdCopter commented Sep 3, 2024

see comment below: #4153 (comment)

Copy link

netlify bot commented Sep 3, 2024

Deploy Preview for origin-betaflight-app ready!

Name Link
🔨 Latest commit 0e974c6
🔍 Latest deploy log https://app.netlify.com/sites/origin-betaflight-app/deploys/66d7387de732700008ef0248
😎 Deploy Preview https://deploy-preview-4153.dev.app.betaflight.com
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@nerdCopter nerdCopter changed the title fast-fail false; do not abort other builds [10.10.1] fast-fail false; do not abort other builds Sep 3, 2024
Copy link

sonarcloud bot commented Sep 3, 2024

Copy link
Contributor

github-actions bot commented Sep 3, 2024

Do you want to test this code? Here you have an automated build:
Betaflight-Configurator-Linux
Betaflight-Configurator-macOS
Betaflight-Configurator-Windows
WARNING: It may be unstable and result in corrupted configurations or data loss. Use only for testing!

@nerdCopter
Copy link
Member Author

nerdCopter commented Sep 3, 2024

This allows other distributions to build if one fails. e.g. here Android fails, but others continue/succeed.
image

I'm also looking into solving Android die to jfrog dependency issue.

edit: failing to solve jfrog issue.
opened #4154

@nerdCopter nerdCopter marked this pull request as ready for review September 3, 2024 16:39
@blckmn blckmn merged commit 866ce2c into betaflight:10.10-maintenance Sep 16, 2024
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants