Skip to content

Use wazero-based libpgquery wrapper in currently disabled environments #3027

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.

Sign up for GitHub

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

Merged
kyleconroy merged 20 commits into from
Dec 5, 2023
Merged

Use wazero-based libpgquery wrapper in currently disabled environments #3027

Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
0bee534
Use wazero-based wrapper of libpgquery when cgo is disabled or on win…
anuraaga Dec 1, 2023
f7c2c9c
Fix
anuraaga Dec 1, 2023
bae1e3c
Fix
anuraaga Dec 1, 2023
f223341
Typo
anuraaga Dec 1, 2023
4009fa4
cgo guard for TestReplay
anuraaga Dec 1, 2023
315cb5a
Windows no test
anuraaga Dec 1, 2023
767e168
Use upstream nodes type
anuraaga Dec 2, 2023
a608c49
Update to latest pg_query_go
anuraaga Dec 5, 2023
ce0fa99
Test matrix
anuraaga Dec 5, 2023
61e9a97
Tweak workflow
anuraaga Dec 5, 2023
d12d574
Fix workflow
anuraaga Dec 5, 2023
e0af7a9
Tweak job name
anuraaga Dec 5, 2023
2d8bb69
Fix tests for nocgo / windows
anuraaga Dec 5, 2023
80881e7
Parallel
anuraaga Dec 5, 2023
03709bb
Clean
anuraaga Dec 5, 2023
918bcba
Fix test
anuraaga Dec 5, 2023
720d2f9
Use windows line ending in test
anuraaga Dec 5, 2023
c2d0f1d
Fix
anuraaga Dec 5, 2023
64078d7
Try more
anuraaga Dec 5, 2023
1ee3272
Ignore e2e on win
anuraaga Dec 5, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Test matrix
  • Loading branch information
@anuraaga
anuraaga committed Dec 5, 2023
commit ce0fa99dcf610106a6e2964300c322b4855dbb93
76 changes: 21 additions & 55 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,61 +5,22 @@ on:
- main
pull_request:
jobs:
windows-build:
if: ${{ github.ref == 'refs/heads/main' }}
runs-on: windows-latest
steps:
- uses: actions/checkout@v4
- uses: actions/setup-go@v4
with:
go-version-file: go.mod
check-latest: true

# Currently some tests rely on line endings so we only build
# without running.
- run: go build ./...

darwin-test:
if: ${{ github.ref == 'refs/heads/main' }}
runs-on: macos-latest
steps:
- uses: actions/checkout@v4
- uses: actions/setup-go@v4
with:
go-version-file: go.mod
check-latest: true

- name: install sqlc-gen-test
run: go install github.com/sqlc-dev/[email protected]

- name: install ./...
run: go install ./...

- run: go test ./...

linux-purego-test:
if: ${{ github.ref == 'refs/heads/main' }}
runs-on: macos-latest
steps:
- uses: actions/checkout@v4
- uses: actions/setup-go@v4
with:
go-version-file: go.mod
check-latest: true

- name: install sqlc-gen-test
run: go install github.com/sqlc-dev/[email protected]

- name: install ./...
run: go install ./...

- run: go test ./...
env:
CGO_ENABLED: '0'

build:
name: test
runs-on: ubuntu-latest
test:
strategy:
max-parallel: 2
matrix:
os: [windows-latest, macos-latest, ubuntu-latest]
cgo: ['1', '0']
# Workaround no proper support for conditional matrix items
# https://github.com/orgs/community/discussions/26253#discussioncomment-6745038
isMain: ${{ github.ref == 'refs/heads/main' }}
exclude:
- { isMain: false }
include:
- { os: ubuntu-latest, cgo: '1' }
- { os: ubuntu-latest, cgo: '0' }
name: Test ${{ matrix.os }}-cgo${{ matrix.cgo }}
runs-on: ${{ matrix.os }}

steps:
- uses: actions/checkout@v4
Expand All @@ -76,17 +37,22 @@ jobs:

- name: install ./...
run: go install ./...
env:
CGO_ENABLED: ${{ matrix.cgo }}

- name: build internal/endtoend
run: go build ./...
working-directory: internal/endtoend/testdata
env:
CGO_ENABLED: ${{ matrix.cgo }}

- name: test ./...
run: gotestsum --junitfile junit.xml -- --tags=examples -timeout 20m ./...
env:
CI_SQLC_PROJECT_ID: ${{ secrets.CI_SQLC_PROJECT_ID }}
CI_SQLC_AUTH_TOKEN: ${{ secrets.CI_SQLC_AUTH_TOKEN }}
SQLC_AUTH_TOKEN: ${{ secrets.CI_SQLC_AUTH_TOKEN }}
CGO_ENABLED: ${{ matrix.cgo }}

vuln_check:
runs-on: ubuntu-latest
Expand Down