Skip to content

Commit

Permalink
Merge branch 'main' into bugfix/custom-context-config-types
Browse files Browse the repository at this point in the history
  • Loading branch information
BrianValente committed May 12, 2024
2 parents dbfed76 + 8817f52 commit c564d55
Show file tree
Hide file tree
Showing 99 changed files with 2,198 additions and 1,424 deletions.
25 changes: 13 additions & 12 deletions .github/workflows/benchmark-parser.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,15 +11,15 @@ jobs:
permissions:
contents: read
outputs:
PR-BENCH-16: ${{ steps.benchmark-pr.outputs.BENCH_RESULT16 }}
PR-BENCH-18: ${{ steps.benchmark-pr.outputs.BENCH_RESULT18 }}
PR-BENCH-20: ${{ steps.benchmark-pr.outputs.BENCH_RESULT20 }}
MAIN-BENCH-16: ${{ steps.benchmark-main.outputs.BENCH_RESULT16 }}
PR-BENCH-22: ${{ steps.benchmark-pr.outputs.BENCH_RESULT22 }}
MAIN-BENCH-18: ${{ steps.benchmark-main.outputs.BENCH_RESULT18 }}
MAIN-BENCH-20: ${{ steps.benchmark-main.outputs.BENCH_RESULT20 }}
MAIN-BENCH-22: ${{ steps.benchmark-main.outputs.BENCH_RESULT22 }}
strategy:
matrix:
node-version: [16, 18, 20]
node-version: [18, 20, 22]
steps:
- uses: actions/checkout@v4
with:
Expand All @@ -33,7 +33,7 @@ jobs:

- name: Install
run: |
npm install --only=production --ignore-scripts
npm install --ignore-scripts
- name: Run benchmark
id: benchmark-pr
Expand All @@ -52,7 +52,7 @@ jobs:

- name: Install
run: |
npm install --only=production --ignore-scripts
npm install --ignore-scripts
- name: Run benchmark
id: benchmark-main
Expand All @@ -64,7 +64,8 @@ jobs:
echo 'EOF' >> $GITHUB_OUTPUT
output-benchmark:
needs: [benchmark]
needs:
- benchmark
runs-on: ubuntu-latest
permissions:
pull-requests: write
Expand All @@ -74,12 +75,6 @@ jobs:
with:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
message: |
**Node**: 16
**PR**: ${{ needs.benchmark.outputs.PR-BENCH-16 }}
**MAIN**: ${{ needs.benchmark.outputs.MAIN-BENCH-16 }}
---
**Node**: 18
**PR**: ${{ needs.benchmark.outputs.PR-BENCH-18 }}
**MAIN**: ${{ needs.benchmark.outputs.MAIN-BENCH-18 }}
Expand All @@ -89,3 +84,9 @@ jobs:
**Node**: 20
**PR**: ${{ needs.benchmark.outputs.PR-BENCH-20 }}
**MAIN**: ${{ needs.benchmark.outputs.MAIN-BENCH-20 }}
---
**Node**: 22
**PR**: ${{ needs.benchmark.outputs.PR-BENCH-22 }}
**MAIN**: ${{ needs.benchmark.outputs.MAIN-BENCH-22 }}
31 changes: 17 additions & 14 deletions .github/workflows/benchmark.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,15 +11,15 @@ jobs:
permissions:
contents: read
outputs:
PR-BENCH-16: ${{ steps.benchmark-pr.outputs.BENCH_RESULT16 }}
PR-BENCH-18: ${{ steps.benchmark-pr.outputs.BENCH_RESULT18 }}
PR-BENCH-20: ${{ steps.benchmark-pr.outputs.BENCH_RESULT20 }}
MAIN-BENCH-16: ${{ steps.benchmark-main.outputs.BENCH_RESULT16 }}
PR-BENCH-22: ${{ steps.benchmark-pr.outputs.BENCH_RESULT22 }}
MAIN-BENCH-18: ${{ steps.benchmark-main.outputs.BENCH_RESULT18 }}
MAIN-BENCH-20: ${{ steps.benchmark-main.outputs.BENCH_RESULT20 }}
MAIN-BENCH-22: ${{ steps.benchmark-main.outputs.BENCH_RESULT22 }}
strategy:
matrix:
node-version: [16, 18, 20]
node-version: [18, 20, 22]
steps:
- uses: actions/checkout@v4
with:
Expand All @@ -33,7 +33,7 @@ jobs:

- name: Install
run: |
npm install --only=production --ignore-scripts
npm install --ignore-scripts
- name: Run benchmark
id: benchmark-pr
Expand All @@ -52,7 +52,7 @@ jobs:

- name: Install
run: |
npm install --only=production --ignore-scripts
npm install --ignore-scripts
- name: Run benchmark
id: benchmark-main
Expand All @@ -64,7 +64,8 @@ jobs:
echo 'EOF' >> $GITHUB_OUTPUT
output-benchmark:
needs: [benchmark]
needs:
- benchmark
runs-on: ubuntu-latest
permissions:
pull-requests: write
Expand All @@ -74,12 +75,6 @@ jobs:
with:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
message: |
**Node**: 16
**PR**: ${{ needs.benchmark.outputs.PR-BENCH-16 }}
**MAIN**: ${{ needs.benchmark.outputs.MAIN-BENCH-16 }}
---
**Node**: 18
**PR**: ${{ needs.benchmark.outputs.PR-BENCH-18 }}
**MAIN**: ${{ needs.benchmark.outputs.MAIN-BENCH-18 }}
Expand All @@ -90,19 +85,27 @@ jobs:
**PR**: ${{ needs.benchmark.outputs.PR-BENCH-20 }}
**MAIN**: ${{ needs.benchmark.outputs.MAIN-BENCH-20 }}
---
**Node**: 22
**PR**: ${{ needs.benchmark.outputs.PR-BENCH-22 }}
**MAIN**: ${{ needs.benchmark.outputs.MAIN-BENCH-22 }}
remove-label:
if: ${{ github.event.label.name == 'benchmark' }}
needs:
needs:
- benchmark
- output-benchmark
runs-on: ubuntu-latest
permissions:
pull-requests: write
steps:
- name: Remove benchmark label
uses: octokit/request-action@v2.x
id: remove-label
with:
route: DELETE /repos/{repo}/issues/{issue_number}/labels/{name}
repo: ${{ github.event.pull_request.head.repo.full_name }}
repo: ${{ github.repository }}
issue_number: ${{ github.event.pull_request.number }}
name: benchmark
env:
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/ci-alternative-runtime.yml
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ jobs:
persist-credentials: false
- uses: nodesource/setup-nsolid@v1
with:
nsolid-version: 5
nsolid-version: 5
- name: install fastify
run: |
npm install --ignore-scripts
Expand All @@ -113,7 +113,7 @@ jobs:
if: >
github.event_name == 'pull_request' &&
github.event.pull_request.user.login == 'dependabot[bot]'
needs:
needs:
- test-typescript
- test-unit
- package
Expand Down
19 changes: 10 additions & 9 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -87,19 +87,21 @@ jobs:
npm run lint
coverage-nix:
needs: lint
needs:
- lint
permissions:
contents: read
uses: ./.github/workflows/coverage-nix.yml

coverage-win:
needs: lint
needs:
- lint
permissions:
contents: read
uses: ./.github/workflows/coverage-win.yml

test-unit:
needs:
needs:
- lint
- coverage-nix
- coverage-win
Expand All @@ -109,12 +111,11 @@ jobs:

strategy:
matrix:
node-version: [14, 16, 18, 20]
node-version: [18, 20, 22]
os: [macos-latest, ubuntu-latest, windows-latest]
exclude:
# excludes node 14 on Windows
- os: windows-latest
node-version: 14
- os: macos-latest
node-version: 16

steps:
- uses: actions/checkout@v4
Expand All @@ -137,7 +138,7 @@ jobs:
npm run unit
test-typescript:
needs:
needs:
- lint
- coverage-nix
- coverage-win
Expand Down Expand Up @@ -203,7 +204,7 @@ jobs:
if: >
github.event_name == 'pull_request' &&
github.event.pull_request.user.login == 'dependabot[bot]'
needs:
needs:
- test-typescript
- test-unit
- package
Expand Down
8 changes: 5 additions & 3 deletions .github/workflows/citgm.yml
Original file line number Diff line number Diff line change
Expand Up @@ -85,18 +85,20 @@ jobs:
node-version: ${{ matrix.node-version }}

remove-label:
if: always()
needs:
if: ${{ github.event.label.name == 'citgm-core-plugins' }}
needs:
- core-plugins
continue-on-error: true
runs-on: ubuntu-latest
permissions:
pull-requests: write
steps:
- name: Remove citgm-core-plugins label
uses: octokit/request-action@v2.x
id: remove-label
with:
route: DELETE /repos/{repo}/issues/{issue_number}/labels/{name}
repo: ${{ github.event.pull_request.head.repo.full_name }}
repo: ${{ github.repository }}
issue_number: ${{ github.event.pull_request.number }}
name: citgm-core-plugins
env:
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/integration-alternative-runtimes.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ jobs:
runs-on: ${{ matrix.os }}
permissions:
contents: read

strategy:
matrix:
os: [ubuntu-latest]
Expand Down Expand Up @@ -49,7 +49,7 @@ jobs:
uses: pnpm/action-setup@v3
with:
version: ${{ matrix.pnpm-version }}

- name: Install Production
run: |
pnpm install --prod
Expand Down
13 changes: 4 additions & 9 deletions .github/workflows/integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,23 +20,18 @@ jobs:
runs-on: ${{ matrix.os }}
permissions:
contents: read

strategy:
matrix:
node-version: [16, 18, 20]
node-version: [18, 20, 22]
os: [ubuntu-latest]
pnpm-version: [8]
# pnpm@8 does not support Node.js 14 so include it separately
include:
- node-version: 14
os: ubuntu-latest
pnpm-version: 7

steps:
- uses: actions/checkout@v4
with:
persist-credentials: false

- name: Use Node.js
uses: actions/setup-node@v4
with:
Expand All @@ -46,7 +41,7 @@ jobs:
uses: pnpm/action-setup@v3
with:
version: ${{ matrix.pnpm-version }}

- name: Install Production
run: |
pnpm install --prod
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/links-check.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ jobs:
# See: https://github.com/lycheeverse/lychee-action/issues/17
- name: Link Checker
id: lychee
uses: lycheeverse/lychee-action@c053181aa0c3d17606addfe97a9075a32723548a
uses: lycheeverse/lychee-action@2b973e86fc7b1f6b36a93795fe2c9c6ae1118621
with:
fail: true
# As external links behavior is not predictable, we check only internal links
Expand Down
13 changes: 4 additions & 9 deletions .github/workflows/package-manager-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,14 +15,9 @@ jobs:
strategy:
matrix:
# Maintenance and active LTS
node-version: [16, 18, 20]
node-version: [18, 20, 22]
os: [ubuntu-latest]
pnpm-version: [8]
# pnpm@8 does not support Node.js 14 so include it separately
include:
- node-version: 14
os: ubuntu-latest
pnpm-version: 7

steps:
- uses: actions/checkout@v4
Expand All @@ -45,15 +40,15 @@ jobs:
run: |
pnpm run test:ci
env:
NODE_OPTIONS: no-network-family-autoselection
NODE_OPTIONS: no-network-family-autoselection

yarn:
runs-on: ${{ matrix.os }}

strategy:
matrix:
# Maintenance and active LTS
node-version: [14, 16, 18, 20]
node-version: [18, 20, 22]
os: [ubuntu-latest]

steps:
Expand All @@ -77,4 +72,4 @@ jobs:
run: |
yarn run test:ci
env:
NODE_OPTIONS: no-network-family-autoselection
NODE_OPTIONS: no-network-family-autoselection
2 changes: 1 addition & 1 deletion .markdownlint-cli2.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ config:
MD013:
line_length: 80
code_block_line_length: 120
headers: false
headings: false
tables: false
strict: false
stern: false
Expand Down
Loading

0 comments on commit c564d55

Please sign in to comment.