From b059dbed630b991d4dc768279d25beaf4f26bf7d Mon Sep 17 00:00:00 2001 From: Guy Ben-Aharon Date: Wed, 13 Aug 2025 15:23:03 +0300 Subject: [PATCH] fix readme --- .github/workflows/test.yml | 12 ++++-------- README.md | 8 ++++---- 2 files changed, 8 insertions(+), 12 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 639d5e8..6dce876 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -62,12 +62,8 @@ jobs: chartdb-api-token: ${{ secrets.TEST_CHARTDB_API_TOKEN || 'test_token' }} chartdb-diagram-id: ${{ secrets.TEST_CHARTDB_DIAGRAM_ID || 'test_diagram' }} - - name: Check for errors in output + - name: Check sync status + if: steps.chartdb-sync-postgres.outputs.status == 'failed' run: | - echo "Checking action output for errors..." - if echo "${{ steps.chartdb-sync-postgres.outputs.* }}" | grep -i "error"; then - echo "Error found in action output!" - exit 1 - else - echo "No errors found in output" - fi + echo "ChartDB sync failed! Check the logs above." + exit 1 diff --git a/README.md b/README.md index 5dd7d89..e78819d 100644 --- a/README.md +++ b/README.md @@ -8,8 +8,8 @@ Automatically sync your database schema with [ChartDB](https://chartdb.io) to ke - 📊 Updates ChartDB diagrams in real-time - 🔒 Secure handling of credentials via GitHub Secrets - 🐳 Docker-based for consistent execution -- 🗄️ Supports PostgreSQL, MySQL, MariaDB, and MSSQL - +- 🗄️ Supports PostgreSQL, MySQL, SQL Server, MariaDB, SQLite, ClickHouse, CockroachDB, and Oracle +x ## Usage ### Basic Example @@ -97,7 +97,7 @@ jobs: | `db-database` | Database name | Yes | - | | `db-username` | Database username | Yes | - | | `db-password` | Database password (use secrets!) | No | - | -| `db-type` | Database type (`postgresql`, `mysql`, `mariadb`, `mssql`) | Yes | - | +| `db-type` | Database type (`postgresql`, `mysql`, `sql_server`, `sqlite`, `mariadb`, `clickhouse`, `cockroachdb`, `oracle`) | Yes | - | | `chartdb-api-token` | ChartDB API token (use secrets!) | Yes | - | | `chartdb-diagram-id` | ChartDB diagram ID to sync with | Yes | - | | `network` | Docker network mode | No | `host` | @@ -165,7 +165,7 @@ db-port: '3306' # default ### Microsoft SQL Server ```yaml -db-type: 'mssql' +db-type: 'sql_server' db-port: '1433' # default ```