Skip to content

fix(docs): install SDK packages in V2 deploy for legacy builder#1433

Merged
hkad98 merged 1 commit intogooddata:masterfrom
hkad98:jkd/fix-netlify-v2-deps
Mar 16, 2026
Merged

fix(docs): install SDK packages in V2 deploy for legacy builder#1433
hkad98 merged 1 commit intogooddata:masterfrom
hkad98:jkd/fix-netlify-v2-deps

Conversation

@hkad98
Copy link
Copy Markdown
Contributor

@hkad98 hkad98 commented Mar 16, 2026

Summary

  • Old release branches don't have griffe_builder.py or method_page_renderer.py in their git tree. The build scripts checked for these files on the target branch before using them, even though they always ran the current branch's copies.

  • For branches without griffe_builder.py, the fallback was json_builder.py which does runtime imports of gooddata_sdk/gooddata_pandas — causing ModuleNotFoundError since those aren't installed in the docs build env.

  • For branches without method_page_renderer.py, Hugo shortcodes (parameters-block, parameter) were left unprocessed. Their templates were removed in the griffe pipeline commit, so Hugo fails with "template for shortcode not found".

Fix: Remove the conditional checks and always use griffe_builder.py (static analysis, no imports) and method_page_renderer.py (replaces shortcodes with inline HTML) from the current branch. Both work on any branch's source code.

fix(docs): install SDK packages in V2 deploy for legacy builder

Older release branches lack griffe_builder.py, so the build falls back to

json_builder.py which imports gooddata_sdk and gooddata_pandas at runtime.

The V2 workflow only installed script-requirements.txt, causing

ModuleNotFoundError for branches using the legacy path.

JIRA: trivial
risk: nonprod
@hkad98 hkad98 enabled auto-merge March 16, 2026 20:11
@hkad98 hkad98 merged commit 01bcdcc into gooddata:master Mar 16, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants