Skip to content

Commit f8d8f9a

Browse files
authored
fix readme (#2)
1 parent 5b26e84 commit f8d8f9a

2 files changed

Lines changed: 8 additions & 12 deletions

File tree

.github/workflows/test.yml

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -62,12 +62,8 @@ jobs:
6262
chartdb-api-token: ${{ secrets.TEST_CHARTDB_API_TOKEN || 'test_token' }}
6363
chartdb-diagram-id: ${{ secrets.TEST_CHARTDB_DIAGRAM_ID || 'test_diagram' }}
6464

65-
- name: Check for errors in output
65+
- name: Check sync status
66+
if: steps.chartdb-sync-postgres.outputs.status == 'failed'
6667
run: |
67-
echo "Checking action output for errors..."
68-
if echo "${{ steps.chartdb-sync-postgres.outputs.* }}" | grep -i "error"; then
69-
echo "Error found in action output!"
70-
exit 1
71-
else
72-
echo "No errors found in output"
73-
fi
68+
echo "ChartDB sync failed! Check the logs above."
69+
exit 1

README.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,8 @@ Automatically sync your database schema with [ChartDB](https://chartdb.io) to ke
88
- 📊 Updates ChartDB diagrams in real-time
99
- 🔒 Secure handling of credentials via GitHub Secrets
1010
- 🐳 Docker-based for consistent execution
11-
- 🗄️ Supports PostgreSQL, MySQL, MariaDB, and MSSQL
12-
11+
- 🗄️ Supports PostgreSQL, MySQL, SQL Server, MariaDB, SQLite, ClickHouse, CockroachDB, and Oracle
12+
x
1313
## Usage
1414

1515
### Basic Example
@@ -97,7 +97,7 @@ jobs:
9797
| `db-database` | Database name | Yes | - |
9898
| `db-username` | Database username | Yes | - |
9999
| `db-password` | Database password (use secrets!) | No | - |
100-
| `db-type` | Database type (`postgresql`, `mysql`, `mariadb`, `mssql`) | Yes | - |
100+
| `db-type` | Database type (`postgresql`, `mysql`, `sql_server`, `sqlite`, `mariadb`, `clickhouse`, `cockroachdb`, `oracle`) | Yes | - |
101101
| `chartdb-api-token` | ChartDB API token (use secrets!) | Yes | - |
102102
| `chartdb-diagram-id` | ChartDB diagram ID to sync with | Yes | - |
103103
| `network` | Docker network mode | No | `host` |
@@ -165,7 +165,7 @@ db-port: '3306' # default
165165

166166
### Microsoft SQL Server
167167
```yaml
168-
db-type: 'mssql'
168+
db-type: 'sql_server'
169169
db-port: '1433' # default
170170
```
171171

0 commit comments

Comments
 (0)