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

Remove cordova logic #4000

Merged
merged 1 commit into from
Jul 24, 2024
Merged

Conversation

chmelevskij
Copy link
Member

Remove traces of cordova in codebase

Depends on:
#3998
#3999

Copy link

netlify bot commented Jun 2, 2024

Deploy Preview for origin-betaflight-app ready!

Name Link
🔨 Latest commit 67a66de
🔍 Latest deploy log https://app.netlify.com/sites/origin-betaflight-app/deploys/66a08f45beeee900084855b6
😎 Deploy Preview https://deploy-preview-4000.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.

@chmelevskij chmelevskij changed the title remove cordova logic Remove cordova logic Jun 2, 2024
Copy link

sonarqubecloud bot commented Jun 2, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
3.9% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

@haslinghuis
Copy link
Member

@chmelevskij please rebase

@chmelevskij chmelevskij force-pushed the remove-cordova-logic branch from b8b6170 to 67a66de Compare July 24, 2024 05:21
Copy link

@nerdCopter
Copy link
Member

  • tested flashing/setup of unsoldered FC.
  • no known side-effects except had CLI tab freeze once, probably nothing to do with this PR.

@haslinghuis haslinghuis merged commit 295e088 into betaflight:master Jul 24, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

3 participants