Skip to content

Commit

Permalink
[#5] Resolve review comments
Browse files Browse the repository at this point in the history
  • Loading branch information
markgravity committed Aug 25, 2023
1 parent d674d6c commit d60ee6e
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 25 deletions.
27 changes: 15 additions & 12 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,17 +50,25 @@ jobs:
run: bundle exec danger

ios_test:
name: Running tests in iOS
name: Running iOS tests
runs-on: macos-latest
defaults:
run:
working-directory: ./sample/ios
steps:
- uses: actions/checkout@v3
- uses: ruby/setup-ruby@v1
- name: Check out
uses: actions/checkout@v3
with:
ruby-version: '3.2'
bundler-cache: true
submodules: recursive

- name: Set up cache
uses: actions/cache@v3
id: bunlderCache
with:
path: vendor/bundle
key: ${{ runner.os }}-gems-${{ hashFiles('**/Gemfile.lock') }}
restore-keys: |
${{ runner.os }}-gems-
- name: Setup Java JDK
uses: actions/[email protected]
Expand Down Expand Up @@ -104,15 +112,10 @@ jobs:
- name: Clean up previous code coverage report
run: bundle exec fastlane cleanUpOutput

- name: Review pull request by Danger
uses: MeilCli/danger-action@v5
with:
plugins_file: 'sample/ios/Gemfile'
install_path: 'sample/ios/vendor/bundle'
danger_file: 'sample/ios/Dangerfile'
danger_id: 'danger-pr'
- name: Run Danger to wrap up the review
env:
DANGER_GITHUB_API_TOKEN: ${{ secrets.GITHUB_TOKEN }}
run: bundle exec danger

generate_sample_project:
# The job "generate_sample_project" identify that the job "test" must complete successfully
Expand Down
2 changes: 1 addition & 1 deletion android/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ plugins {
kotlin(Plugins.ANDROID)
}

val keystoreProperties = rootDir.loadGradleProperties("signing.properties")
val keystoreProperties = loadProperties("$rootDir/signing.properties")

android {
namespace = "co.nimblehq.kmm.template.android"
Expand Down
12 changes: 0 additions & 12 deletions buildSrc/src/main/java/FileExt.kt

This file was deleted.

0 comments on commit d60ee6e

Please sign in to comment.