-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
How to turn off Invalid Warning check? #40
Comments
@zgosalvez would love to hear if you got a workaround |
This is our workflow btw: name: Run checks
on:
pull_request:
types:
- opened
- reopened
- synchronize
- ready_for_review
env:
FLUTTER_VERSION: "3.13.9"
FLUTTER_CHANNEL: "stable"
jobs:
build:
name: Run checks
runs-on: ubuntu-latest
if: ${{ github.event.pull_request.draft == false }}
steps:
- name: Checkout code
uses: actions/checkout@v2
- name: Set up Flutter
uses: subosito/[email protected]
with:
flutter-version: ${{ env.FLUTTER_VERSION }}
channel: ${{ env.FLUTTER_CHANNEL }}
architecture: x64
cache: true
- name: Create .env
run: echo "WHY_CANNOT_I_BE_EMPTY" > .env
- run: |
flutter pub get
flutter pub run build_runner build --delete-conflicting-outputs
- name: Analyze Dart
uses: zgosalvez/github-actions-analyze-dart@v3
with:
line-length: 150
fail-on-warnings: true
fail-on-infos: false |
Perhaps it could be because line-length is not respected? |
I'm curious to know why you'd like to turn off warnings. By design, this action is informative based on the output reported by Dart. By default, this action won't fail anyway since |
@brodienguyen / @simplenotezy, care to verify if the PR will resolve your issue? If so, I will merge the changes. |
@zgosalvez Hi, Thank you very much for providing this great Action first! This PR([#96) is useful because generated files doesn't and shouldn't be formatted with These files also configured to be excluded in I'm also feel confused by these warnings thrown from my |
Hi there,
Please let me know if there is a way to skip this warning message. Btw, I wonder why it raises for those import lines, please?
Regards,
Brodie
The text was updated successfully, but these errors were encountered: