Skip to content

Commit

Permalink
Merge branch 'main' into feature/sc-45636/sdk-make-a-svelte-react-imp…
Browse files Browse the repository at this point in the history
…lementation-that
  • Loading branch information
etienneburdet committed Jan 24, 2024
2 parents 5814d8a + ba4f50b commit 366ec7b
Show file tree
Hide file tree
Showing 22 changed files with 74,451 additions and 75,036 deletions.
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
name: api-client coverage
on:
pull_request:
push:
paths:
- 'packages/api-client/**'
jobs:
Expand All @@ -25,13 +25,17 @@ jobs:
- name: Install
run: npm ci
working-directory: ./packages/api-client

- name: test
run: npm run test
working-directory: ./packages/api-client

- name: Coverage
run: npm run test:coverage
working-directory: ./packages/api-client

- name: Comment
uses: romeovs/[email protected].16
uses: romeovs/[email protected].19
with:
github-token: ${{ secrets.GITHUB_TOKEN }}
lcov-file: ./packages/api-client/coverage/lcov.info
lcov-file: ./packages/api-client/coverage/lcov.info
39 changes: 0 additions & 39 deletions .github/workflows/chromatic.yml

This file was deleted.

29 changes: 0 additions & 29 deletions .github/workflows/lint.yml

This file was deleted.

29 changes: 0 additions & 29 deletions .github/workflows/test.yml

This file was deleted.

104 changes: 104 additions & 0 deletions .github/workflows/visualizations.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,104 @@
name: "Visualizations"

on:
pull_request:
types:
- opened
- synchronize
- reopened
- ready_for_review
paths-ignore:
- "packages/api-client/**"

jobs:
test:
name: Build and test on Node ${{ matrix.node }} and ${{ matrix.os }}

runs-on: ${{ matrix.os }}
strategy:
matrix:
node: ["14.x", "16.17.1"]
os: [ubuntu-latest]

steps:
- name: Checkout repo
uses: actions/checkout@v2

- name: Use Node ${{ matrix.node }}
uses: actions/setup-node@v1
with:
node-version: ${{ matrix.node }}

- name: Install
run: npm ci

- name: Bootstrap
run: npm run bootstrap

- name: Test
run: npm test

lint:
name: Lint on Node ${{ matrix.node }} and ${{ matrix.os }}

runs-on: ${{ matrix.os }}
strategy:
matrix:
node: ["16.17.1"]
os: [ubuntu-latest]

steps:
- name: Checkout repo
uses: actions/checkout@v2

- name: Use Node ${{ matrix.node }}
uses: actions/setup-node@v1
with:
node-version: ${{ matrix.node }}

- name: Install
run: npm ci

- name: Bootstrap
run: npm run bootstrap

- name: Lint
run: npm run lint

chromatic-deployment:
if: github.event.pull_request.draft == false
runs-on: ubuntu-latest
strategy:
matrix:
node: [">=16 <18"]

needs: [test, lint]

steps:
- name: Checkout repo
uses: actions/checkout@v2
with:
fetch-depth: 0

- name: Use Node ${{ matrix.node }}
uses: actions/setup-node@v1
with:
node-version: ${{ matrix.node }}

- name: Install
run: npm ci

- name: Bootstrap
run: npm run bootstrap

- name: Build storybook
run: npm run build-storybook
working-directory: ./packages/visualizations-react

- name: Publish to Chromatic
uses: chromaui/action@v1
with:
token: ${{ secrets.GITHUB_TOKEN }}
projectToken: ${{ secrets.CHROMATIC_TOKEN }}
workingDir: ./packages/visualizations-react
storybookBuildDir: storybook-static
11 changes: 11 additions & 0 deletions packages/api-client/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,17 @@
All notable changes to this project will be documented in this file.
See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.

# [21.2.0](https://github.com/opendatasoft/ods-dataviz-sdk/compare/@opendatasoft/[email protected]...@opendatasoft/[email protected]) (2024-01-10)


### Features

* **api-client:** Add new api client option: headers ([985f883](https://github.com/opendatasoft/ods-dataviz-sdk/commit/985f8839ccb857b60f35bdc1187558cb018a2a95))





## [21.1.2](https://github.com/opendatasoft/ods-dataviz-sdk/compare/@opendatasoft/[email protected]...@opendatasoft/[email protected]) (2023-10-12)


Expand Down
Loading

1 comment on commit 366ec7b

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverage for this commit

95.21%

Coverage Report
FileBranchesFuncsLinesUncovered Lines
src
   index.ts100%100%100%
src/client
   error.ts100%100%100%
   index.ts75.32%100%96.88%124, 13, 146, 148, 148–149, 15, 15, 151, 162, 169, 169, 17, 17, 171, 176, 179, 182, 184, 52, 82
   types.ts100%100%100%
src/odsql
   clauses.ts71.43%80%90.91%14, 32, 42
   index.ts83.72%95.65%94.12%111, 140, 25, 28, 56–57, 57, 57–58, 68, 78–79

Please sign in to comment.