Skip to content

Commit

Permalink
Add actions
Browse files Browse the repository at this point in the history
  • Loading branch information
Rexios80 committed Sep 10, 2024
1 parent 4807d66 commit be4581f
Show file tree
Hide file tree
Showing 3 changed files with 35 additions and 1 deletion.
33 changes: 33 additions & 0 deletions .github/workflows/analysis.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,33 @@
name: Analyze
on:
pull_request:
push:
branches:
- main

concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: true

jobs:
analyze-dart:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: subosito/flutter-action@v2
with:
cache: true
- name: Pub cache
uses: actions/cache@v4
env:
CACHE_NAME: firebase_js_interop
with:
path: ${{ env.PUB_CACHE }}
key: pub-cache-${{ env.CACHE_NAME }}-${{ hashFiles('**/pubspec.lock') }}
restore-keys: pub-cache-${{ env.CACHE_NAME }}-
- name: Pub get
run: flutter pub get
- name: Format
run: dart format --set-exit-if-changed .
- name: Analyze
run: dart analyze --fatal-infos .
1 change: 1 addition & 0 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ The process for adding new interop code is as follows:
## Style

- Only write interop for classes and functions that are public, non-deprecated, and meant for external use
- Maintain the same declaration order as the JS SDK
- Doc comments should be verbatim copied from the JS SDK, however convert them from `*` to `///`
- Comment the expected signature of `JSFunction`s. See existing interop code for examples.
- If there are multiple function definitions with the same name, choose the most flexible one
Expand Down
2 changes: 1 addition & 1 deletion example/fji_example/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ publish_to: none
version: 1.0.0+1

environment:
sdk: ^3.6.0-149.3.beta
sdk: ^3.0.0

dependencies:
flutter:
Expand Down

0 comments on commit be4581f

Please sign in to comment.