Skip to content

fix(tokens): commit dist/ and guard prepare script for F-Droid compat…#575

Merged
najuna-brian merged 1 commit intoOpenDataEnsemble:devfrom
najuna-brian:fix/tokens-commit-dist
Mar 26, 2026
Merged

fix(tokens): commit dist/ and guard prepare script for F-Droid compat…#575
najuna-brian merged 1 commit intoOpenDataEnsemble:devfrom
najuna-brian:fix/tokens-commit-dist

Conversation

@najuna-brian
Copy link
Contributor

Fix: Description

Fixes the F-Droid build failure where style-dictionary (a devDependency of @ode/tokens) wasn't available when tokens was consumed as a file: dependency during npm ci.

Changes

  • Commit pre-built dist/ directory so consumers don't need to build tokens
  • Guard prepare script: only runs build if dist/ doesn't exist
  • Un-gitignore dist/ in packages/tokens

Benefits

  • No more mandatory cd packages/tokens && npm install after cloning
  • F-Droid CI can build without needing style-dictionary installed
  • npm run build still works for rebuilding after token changes

@najuna-brian najuna-brian requested a review from r0ssing March 25, 2026 21:26
@najuna-brian najuna-brian merged commit 913a858 into OpenDataEnsemble:dev Mar 26, 2026
11 checks passed
@najuna-brian najuna-brian deleted the fix/tokens-commit-dist branch March 26, 2026 08:43
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