Skip to content

build: upgrade pnpm to 9.7.0 and abandon yarn #344

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

Merged
merged 4 commits into from
Aug 9, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
3 changes: 0 additions & 3 deletions .czrc

This file was deleted.

113 changes: 56 additions & 57 deletions .github/ISSUE_TEMPLATE/1_bug_report.yaml
Original file line number Diff line number Diff line change
@@ -1,67 +1,66 @@
name: Bug Report
description: File a bug report
title: "[Bug]: "
title: '[Bug]: '
labels: []
body:
- type: markdown
attributes:
value: |
Thanks for taking the time to fill out this bug report!
- type: markdown
attributes:
value: |
Thanks for taking the time to fill out this bug report!

- type: input
id: version
attributes:
label: Version
description: Which version did the problem appear on?
placeholder: v0.0.1
validations:
required: true
- type: input
id: version
attributes:
label: Version
description: Which version did the problem appear on?
placeholder: v0.0.1
validations:
required: true

- type: dropdown
id: sql-type
attributes:
label: Which SQL?
description: Which kind of SQL did the problem appear on?
options:
- MySQL
- Flink
- Spark
- Hive
- Postgre
- Trino
- Impala
- Any Others
validations:
required: true
- type: dropdown
id: sql-type
attributes:
label: Which SQL?
description: Which kind of SQL did the problem appear on?
options:
- MySQL
- Flink
- Spark
- Hive
- Postgre
- Trino
- Impala
- Any Others
validations:
required: true

- type: textarea
id: sql-content
attributes:
label: SQL content
placeholder: SELECT * FROM table;
description: Please provide the minimal SQL text that reproduces the problem.
render: sql
- type: textarea
id: sql-content
attributes:
label: SQL content
placeholder: SELECT * FROM table;
description: Please provide the minimal SQL text that reproduces the problem.
render: sql

- type: textarea
id: js-code
attributes:
label: JavaScript/TypeScript code
description: Please provide the minimal demo code that reproduces the problem.
render: typescript
- type: textarea
id: js-code
attributes:
label: JavaScript/TypeScript code
description: Please provide the minimal demo code that reproduces the problem.
render: typescript

- type: textarea
id: what-happened
attributes:
label: What happened?
description: Also tell us, what did you expect to happen?
placeholder: Tell us what you see!
validations:
required: true

- type: textarea
id: logs
attributes:
label: Relevant log output
description: Please copy and paste any relevant log output. This will be automatically formatted into code, so no need for backticks.
render: shell
- type: textarea
id: what-happened
attributes:
label: What happened?
description: Also tell us, what did you expect to happen?
placeholder: Tell us what you see!
validations:
required: true

- type: textarea
id: logs
attributes:
label: Relevant log output
description: Please copy and paste any relevant log output. This will be automatically formatted into code, so no need for backticks.
render: shell
46 changes: 23 additions & 23 deletions .github/ISSUE_TEMPLATE/2_feature_request.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,28 +3,28 @@ description: Suggest an idea for this project
title: '[Feature Request]: '
labels: []
body:
- type: markdown
attributes:
value: |
To help us efficiently reviewing your feature request, please fill out this form.
- type: markdown
attributes:
value: |
To help us efficiently reviewing your feature request, please fill out this form.

- type: dropdown
id: sql-type
attributes:
label: Topic
options:
- Support new SQL
- Support new features on parser
- Enhance existing features
- Documents
- Any Others
validations:
required: true
- type: dropdown
id: sql-type
attributes:
label: Topic
options:
- Support new SQL
- Support new features on parser
- Enhance existing features
- Documents
- Any Others
validations:
required: true

- type: textarea
id: description
attributes:
label: Description
description: Please describe your feature request.
validations:
required: true
- type: textarea
id: description
attributes:
label: Description
description: Please describe your feature request.
validations:
required: true
98 changes: 98 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,98 @@
# This workflow will do a clean installation of node dependencies, cache/restore them, build the source code and run tests across different versions of node
# For more information see: https://docs.github.com/en/actions/automating-builds-and-tests/building-and-testing-nodejs

name: Node.js CI

on:
push:
branches: [main, next]
pull_request:
branches: [main, next]

jobs:
check:
runs-on: ubuntu-latest
strategy:
matrix:
node-version: [18.x, 20.x]
# See supported Node.js release schedule at https://nodejs.org/en/about/releases/

steps:
- name: Checkout
uses: actions/checkout@v4

- name: Install pnpm
uses: pnpm/[email protected]
with:
version: 9.7.0

- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node-version }}
cache: 'pnpm'

- name: Install dependencies
run: pnpm install --frozen-lockfile

- name: Run prettier check
run: pnpm prettier-check

- name: Run tsc check
run: pnpm check-types

test:
runs-on: ubuntu-latest
strategy:
matrix:
node-version: [18.x, 20.x]
# See supported Node.js release schedule at https://nodejs.org/en/about/releases/

steps:
- name: Checkout
uses: actions/checkout@v4

- name: Install pnpm
uses: pnpm/[email protected]
with:
version: 9.7.0

- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node-version }}
cache: 'pnpm'

- name: Install dependencies
run: pnpm install --frozen-lockfile

- name: Run Units Test
run: pnpm test

build:
runs-on: ubuntu-latest
strategy:
matrix:
node-version: [18.x, 20.x]
# See supported Node.js release schedule at https://nodejs.org/en/about/releases/

steps:
- name: Checkout
uses: actions/checkout@v4

- name: Install pnpm
uses: pnpm/[email protected]
with:
version: 9.7.0

- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node-version }}
cache: 'pnpm'

- name: Install dependencies
run: pnpm install --frozen-lockfile

- name: Run Build
run: pnpm build
Loading