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

theme check fails, but action passes #5

Closed
Morgy93 opened this issue Oct 25, 2021 · 1 comment
Closed

theme check fails, but action passes #5

Morgy93 opened this issue Oct 25, 2021 · 1 comment

Comments

@Morgy93
Copy link

Morgy93 commented Oct 25, 2021

I just ran this GitHub action on a fresh Shopify/dawn v2.3.0 and the theme-check throws this Shopify/dawn#686 error, but the check passes nonetheless:

image

Maybe that's the cause why it got in the release in the first place?

I'd assume that the theme-check-action does not "catch" any errors then.

@charlespwd
Copy link
Contributor

It's because the default error level is "error". Suggestions are like warnings, you can pass CI without them.

If you want to change this behaviour, you can set the flags option to the following:

# .github/workflows/theme-check.yml
name: Theme Check
on: [push]
jobs:
  theme-check:
    name: Theme Check
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2
      - name: Theme Check
        uses: shopify/theme-check-action@v1
        with:
          theme_root: '.'
          flags: '--fail-level suggestion' # this means run theme-check --fail-level suggestion

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants