Skip to content

Commit

Permalink
Test v1.1.0 and v1.2.0 (#38)
Browse files Browse the repository at this point in the history
* Test with sqlfluff 1.1.0 and 1.2.0

* Test
  • Loading branch information
yu-iskw authored Jul 15, 2022
1 parent 110ca54 commit a856d3a
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 13 deletions.
8 changes: 4 additions & 4 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,9 @@ jobs:
fail-fast: false
matrix:
sqlfluff:
- "0.12.0"
- "0.13.1"
- "1.0.0"
- "1.1.0"
- "1.2.0"
config:
- ".sqlfluff.bigquery"
- ".sqlfluff.postgres"
Expand Down Expand Up @@ -55,9 +55,9 @@ jobs:
fail-fast: false
matrix:
sqlfluff:
- "0.12.0"
- "0.13.1"
- "1.0.0"
- "1.1.0"
- "1.2.0"
config:
- ".sqlfluff.bigquery"
- ".sqlfluff.postgres"
Expand Down
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
setup-dev:
SQLFLUFF_VERSION=v0.10.1 pip install -r requirements/requirements.txt
SQLFLUFF_VERSION=1.2.0 pip install -r requirements/requirements.txt
10 changes: 5 additions & 5 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ jobs:
with:
github_token: ${{ secrets.github_token }}
reporter: github-pr-review
sqlfluff_version: "0.13.0"
sqlfluff_version: "1.2.0"
sqlfluff_command: "fix" # Or "lint"
config: "${{ github.workspace }}/.sqlfluff"
paths: '${{ github.workspace }}/models'
Expand All @@ -54,8 +54,9 @@ jobs:
## NOTE
The tested sqlfluff versions in the repositories are:
- 0.12.0
- 0.13.1
- 1.0.0
- 1.1.0
- 1.2.0
## Input
Expand Down Expand Up @@ -98,9 +99,8 @@ inputs:
sqlfluff_version:
description: |
sqlfluff version. Use the latest version if not set.
It must be 0.9.0 or later, because `sqlfluff<=0.8.2` doesn't support the `--config` option in the `fix` sub command.
required: false
default: '0.9.4'
default: '1.2.0'
sqlfluff_command:
description: 'The sub command of sqlfluff. One of lint and fix'
required: false
Expand Down
3 changes: 1 addition & 2 deletions action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,9 +43,8 @@ inputs:
sqlfluff_version:
description: |
sqlfluff version. Use the latest version if not set.
It must be 0.9.0 or later, because `sqlfluff<=0.8.2` doesn't support the `--config` option in the `fix` sub command.
required: false
default: '1.0.0'
default: '1.2.0'
sqlfluff_command:
description: 'The sub command of sqlfluff. One of lint and fix'
required: false
Expand Down
2 changes: 1 addition & 1 deletion testdata/test_failed_dbt/models/staging/staging_test.sql
Original file line number Diff line number Diff line change
Expand Up @@ -2,5 +2,5 @@

SELECT
1 AS x,
2 AS y,
2 AS y,
3 AS z

0 comments on commit a856d3a

Please sign in to comment.