Skip to content

fix: missing dependencies and tailwind config errors#165

Open
lixun910 wants to merge 2 commits intomainfrom
xli-fix-build-and-style
Open

fix: missing dependencies and tailwind config errors#165
lixun910 wants to merge 2 commits intomainfrom
xli-fix-build-and-style

Conversation

@lixun910
Copy link
Collaborator

@lixun910 lixun910 commented Oct 14, 2025

  • add some missing dependencies from some packages
  • fix tailwind config (probably misconfigured by llm)

It is also interesting that github actions passed build check. I think maybe caused by the cache?

      - name: Setup Node
        uses: actions/setup-node@v4
        with:
          node-version: 22
          cache: 'pnpm'

@lixun910 lixun910 requested a review from ilyabo October 14, 2025 23:34
@@ -8,6 +8,7 @@ const config = {
// If you make a precise list of packages used, instead of @sqlrooms/*,
// it would help Vite start faster in dev mode
'{./,../../}node_modules/@sqlrooms/*/dist/**/*.js',
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe we should just try to change it to everywhere?

'**/node_modules/@sqlrooms/*/dist/**/*.js'

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