Skip to content

Commit

Permalink
fix: compile and CI
Browse files Browse the repository at this point in the history
  • Loading branch information
timbru31 committed Dec 7, 2021
1 parent ca8816e commit 9e0a4cc
Show file tree
Hide file tree
Showing 5 changed files with 4 additions and 9 deletions.
1 change: 0 additions & 1 deletion .github/workflows/smoke-test-macos.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,6 @@ jobs:
gsed -i '4i\ <preference name="SwiftVersion" value="5" />' config.xml
cordova platform add android@latest
gsed -i '4i\ <preference name="GradlePluginKotlinEnabled" value="true" />' config.xml
gsed -i '4i\ <preference name="AndroidXEnabled" value="true" />' config.xml
- name: Install Plugins
run: |
Expand Down
5 changes: 2 additions & 3 deletions .github/workflows/smoke-test-ubuntu-ionic.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,13 +31,12 @@ jobs:
npm i -g cordova @ionic/cli
ionic start lottie-smoke-test tabs --cordova --type angular --no-interactive --no-confirm
cd lottie-smoke-test
ionic cordova platform add android@9.1.0 --no-interactive --confirm
ionic cordova platform add android@latest --no-interactive --confirm
sed -i '4i\ <preference name="GradlePluginKotlinEnabled" value="true" />' config.xml
- name: Install Plugins
run: |
cd lottie-smoke-test
ionic cordova plugin add cordova-plugin-androidx --no-interactive --confirm
ionic cordova plugin add cordova-plugin-androidx-adapter --no-interactive --confirm
ionic cordova plugin add $GITHUB_WORKSPACE --no-interactive --confirm -- --link
- name: Build Project
Expand Down
1 change: 0 additions & 1 deletion .github/workflows/smoke-test-ubuntu.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,6 @@ jobs:
cd lottie-smoke-test
cordova platform add android@latest
sed -i '4i\ <preference name="GradlePluginKotlinEnabled" value="true" />' config.xml
sed -i '4i\ <preference name="AndroidXEnabled" value="true" />' config.xml
- name: Install Plugins
run: |
Expand Down
4 changes: 1 addition & 3 deletions .github/workflows/smoke-test-windows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ jobs:

- uses: actions/setup-node@v2.5.0
with:
node-version: '16.x'
node-version: '12.x'

- name: Environment Information
run: |
Expand All @@ -35,8 +35,6 @@ jobs:
- name: Install Plugins
run: |
cd lottie-smoke-test
cordova plugin add cordova-plugin-androidx
cordova plugin add cordova-plugin-androidx-adapter
cordova plugin add $env:GITHUB_WORKSPACE --link
- name: Build Project
Expand Down
2 changes: 1 addition & 1 deletion src/ios/LottieSplashScreen.swift
Original file line number Diff line number Diff line change
Expand Up @@ -268,7 +268,7 @@ import Lottie
)
}

private func getUIModeDependentPreference(basePreferenceName: String, defaultValue: String? = "") -> String {
private func getUIModeDependentPreference(basePreferenceName: String, defaultValue: String = "") -> String {
var preferenceValue = ""
if #available(iOS 12.0, *) {
if viewController.traitCollection.userInterfaceStyle == .dark {
Expand Down

0 comments on commit 9e0a4cc

Please sign in to comment.