This is an automated email from the ASF dual-hosted git repository.

rusackas pushed a commit to branch feat/db-engine-docs
in repository https://gitbox.apache.org/repos/asf/superset.git

commit 6116b8383fdcdee83017485de53bdb09a48b553d
Author: Evan Rusackas <[email protected]>
AuthorDate: Mon Dec 22 23:03:31 2025 -0800

    fix(docs): make README update opt-in to avoid CI issues
    
    The generate-database-docs script now only updates README.md when
    explicitly requested via:
    - --update-readme flag
    - UPDATE_README=true env var
    
    Added npm script: yarn update:readme-db-logos
    
    This prevents CI from failing due to uncommitted README changes
    during docs builds.
    
    🤖 Generated with [Claude Code](https://claude.com/claude-code)
    
    Co-Authored-By: Claude Opus 4.5 <[email protected]>
---
 docs/package.json                       | 1 +
 docs/scripts/generate-database-docs.mjs | 8 +++++---
 docs/src/data/databases.json            | 2 +-
 3 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/docs/package.json b/docs/package.json
index b699f81da1..071423e988 100644
--- a/docs/package.json
+++ b/docs/package.json
@@ -18,6 +18,7 @@
     "typecheck": "yarn run generate:extension-components && tsc",
     "generate:extension-components": "node 
scripts/generate-extension-components.mjs",
     "generate:database-docs": "node scripts/generate-database-docs.mjs",
+    "update:readme-db-logos": "node scripts/generate-database-docs.mjs 
--update-readme",
     "eslint": "eslint .",
     "version:add": "node scripts/manage-versions.mjs add",
     "version:remove": "node scripts/manage-versions.mjs remove",
diff --git a/docs/scripts/generate-database-docs.mjs 
b/docs/scripts/generate-database-docs.mjs
index 83d393e311..afa50af373 100644
--- a/docs/scripts/generate-database-docs.mjs
+++ b/docs/scripts/generate-database-docs.mjs
@@ -712,9 +712,11 @@ async function main() {
   );
   console.log(`  Generated _category_.json`);
 
-  // Update README.md database logos
-  console.log('');
-  updateReadme(databases);
+  // Update README.md database logos (only when explicitly requested)
+  if (process.env.UPDATE_README === 'true' || 
process.argv.includes('--update-readme')) {
+    console.log('');
+    updateReadme(databases);
+  }
 
   console.log(`\nStatistics:`);
   console.log(`  Total databases: ${statistics.totalDatabases}`);
diff --git a/docs/src/data/databases.json b/docs/src/data/databases.json
index c03695d856..caba45789f 100644
--- a/docs/src/data/databases.json
+++ b/docs/src/data/databases.json
@@ -1,5 +1,5 @@
 {
-  "generated": "2025-12-22T20:10:13.023Z",
+  "generated": "2025-12-23T07:03:17.267Z",
   "statistics": {
     "totalDatabases": 53,
     "withDocumentation": 53,

Reply via email to