Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Version Packages #725

Merged
merged 1 commit into from
Feb 17, 2025
Merged

Version Packages #725

merged 1 commit into from
Feb 17, 2025

Conversation

liam-changeset-ci-trigger[bot]
Copy link

@liam-changeset-ci-trigger liam-changeset-ci-trigger bot commented Feb 14, 2025

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.

Releases

@liam-hq/[email protected]

Minor Changes

  • #739 - ✨ Pre-calculate layout for Related Tables before displaying in Main Area / Thanks @junkisai!

Patch Changes

  • #708 - ♻️ Swap useReactflow's fitView() with a custom fitView() function / Thanks @junkisai!
  • #708 - ♻️ Remove useSyncHiddenNodesChange hook and update VisibilityButton to manage node visibility directly / Thanks @junkisai!
  • #708 - ♻️ Consolidate table selection/deselection logic into useTableSelection hook / Thanks @junkisai!
  • #708 - 🐛 Fix RelatedTables additional buttons prop rendering / Thanks @junkisai!
  • #708 - ♻️ Limit useAutoLayout functionality to layout calculations only / Thanks @junkisai!
  • #726 - feat: expose all useReactflow functions in useCustomReactflow and replace direct useReactflow usages / Thanks @devin-ai-integration!
  • #739 - ✨ Pre-calculate layout for Related Tables before displaying in Main Area / Thanks @junkisai!
  • #708 - ♻️ Swap useReactflow's fitView() with a custom fitView() function / Thanks @junkisai!
  • #708 - ♻️ Remove useSyncHiddenNodesChange hook and update VisibilityButton to manage node visibility directly / Thanks @junkisai!
  • #708 - ♻️ Consolidate table selection/deselection logic into useTableSelection hook / Thanks @junkisai!
  • #708 - 🐛 Fix RelatedTables additional buttons prop rendering / Thanks @junkisai!
  • #708 - ♻️ Limit useAutoLayout functionality to layout calculations only / Thanks @junkisai!

@liam-hq/[email protected]

Minor Changes

  • #739 - ✨ Pre-calculate layout for Related Tables before displaying in Main Area / Thanks @junkisai!

Patch Changes

  • #726 - feat: expose all useReactflow functions in useCustomReactflow and replace direct useReactflow usages / Thanks @devin-ai-integration!
  • #708 - ♻️ Swap useReactflow's fitView() with a custom fitView() function / Thanks @junkisai!
  • #708 - ♻️ Remove useSyncHiddenNodesChange hook and update VisibilityButton to manage node visibility directly / Thanks @junkisai!
  • #708 - ♻️ Consolidate table selection/deselection logic into useTableSelection hook / Thanks @junkisai!
  • #708 - 🐛 Fix RelatedTables additional buttons prop rendering / Thanks @junkisai!
  • #708 - ♻️ Limit useAutoLayout functionality to layout calculations only / Thanks @junkisai!

@liam-changeset-ci-trigger liam-changeset-ci-trigger bot requested a review from a team as a code owner February 14, 2025 05:15
@liam-changeset-ci-trigger liam-changeset-ci-trigger bot requested review from hoshinotsuyoshi, FunamaYukina, junkisai, MH4GF and sasamuku and removed request for a team February 14, 2025 05:15
Copy link

vercel bot commented Feb 14, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

3 Skipped Deployments
Name Status Preview Comments Updated (UTC)
test-liam-docs ⬜️ Ignored (Inspect) Visit Preview Feb 17, 2025 3:26am
test-liam-erd-sample ⬜️ Ignored (Inspect) Visit Preview Feb 17, 2025 3:26am
test-liam-erd-web ⬜️ Ignored (Inspect) Feb 17, 2025 3:26am

Copy link
Contributor

qodo-merge-pro-for-open-source bot commented Feb 14, 2025

CI Feedback 🧐

(Feedback updated until commit 4c99768)

A test triggered by this PR failed. Here is an AI-generated analysis of the failure:

Action: run-e2e / e2e-test (Mobile Safari)

Failed stage: Run e2e tests [❌]

Failed test name: tests/vrt/vrt.test.ts:21:5 › top

Failure summary:

The action failed due to two test failures:
1. Visual Regression Test (VRT) failure in Mobile Safari
for the 'top' test. The screenshot comparison failed after 3 attempts (original + 2 retries),
indicating visual differences between expected and actual renders.
2. E2E test failure in Mobile
Safari for the 'Table node should be highlighted when clicked' test. The test expected a CSS class
table-node-highlighted but received a different value.

Relevant error logs:
1:  ##[group]Operating System
2:  Ubuntu
...

195:  Scope: all 11 workspace projects
196:  Lockfile is up to date, resolution step is skipped
197:  Progress: resolved 1, reused 0, downloaded 0, added 0
198:  Packages: +1440
199:  ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
200:  Progress: resolved 1440, reused 1370, downloaded 0, added 0
201:  Progress: resolved 1440, reused 1420, downloaded 0, added 618
202:  Progress: resolved 1440, reused 1420, downloaded 0, added 1440, done
203:  WARN  Failed to create bin at /home/runner/work/liam/liam/frontend/apps/erd-sample/node_modules/.bin/liam. ENOENT: no such file or directory, open '/home/runner/work/liam/liam/frontend/packages/cli/dist-cli/bin/cli.js'
...

208:  + @turbo/gen 2.1.2
209:  + syncpack 13.0.0
210:  + turbo 2.1.2
211:  frontend/apps/docs postinstall$ fumadocs-mdx
212:  frontend/apps/docs postinstall: [MDX] types generated
213:  frontend/apps/docs postinstall: Done
214:  frontend/apps/erd-web postinstall$ cp ../../packages/db-structure/node_modules/@ruby/prism/src/prism.wasm prism.wasm
215:  frontend/apps/erd-web postinstall: Done
216:  WARN  Failed to create bin at /home/runner/work/liam/liam/frontend/apps/erd-sample/node_modules/.bin/liam. ENOENT: no such file or directory, open '/home/runner/work/liam/liam/frontend/apps/erd-sample/node_modules/@liam-hq/cli/dist-cli/bin/cli.js'
...

1368:  env:
1369:  CI: true
1370:  PNPM_HOME: /home/runner/setup-pnpm/node_modules/.bin
1371:  URL: https://liam-erd-nn4gcdhqr-route-06-core.vercel.app
1372:  ##[endgroup]
1373:  Running 17 tests using 1 worker
1374:  °°°·°×±·····°°°°°××F
1375:  1) [Mobile Safari] › tests/vrt/vrt.test.ts:21:5 › top ────────────────────────────────────────────
1376:  Error: �[2mexpect(�[22m�[31mpage�[39m�[2m).�[22mtoHaveScreenshot�[2m(�[22m�[32mexpected�[39m�[2m)�[22m
...

1407:  ────────────────────────────────────────────────────────────────────────────────────────────────
1408:  attachment #2: top-1-actual.png (image/png) ────────────────────────────────────────────────────
1409:  test-results/vrt-vrt-top-Mobile-Safari/top-1-actual.png
1410:  ────────────────────────────────────────────────────────────────────────────────────────────────
1411:  attachment #3: top-1-diff.png (image/png) ──────────────────────────────────────────────────────
1412:  test-results/vrt-vrt-top-Mobile-Safari/top-1-diff.png
1413:  ────────────────────────────────────────────────────────────────────────────────────────────────
1414:  Retry #1 ───────────────────────────────────────────────────────────────────────────────────────
1415:  Error: �[2mexpect(�[22m�[31mpage�[39m�[2m).�[22mtoHaveScreenshot�[2m(�[22m�[32mexpected�[39m�[2m)�[22m
...

1451:  test-results/vrt-vrt-top-Mobile-Safari-retry1/top-1-diff.png
1452:  ────────────────────────────────────────────────────────────────────────────────────────────────
1453:  attachment #4: trace (application/zip) ─────────────────────────────────────────────────────────
1454:  test-results/vrt-vrt-top-Mobile-Safari-retry1/trace.zip
1455:  Usage:
1456:  pnpm exec playwright show-trace test-results/vrt-vrt-top-Mobile-Safari-retry1/trace.zip
1457:  ────────────────────────────────────────────────────────────────────────────────────────────────
1458:  Retry #2 ───────────────────────────────────────────────────────────────────────────────────────
1459:  Error: �[2mexpect(�[22m�[31mpage�[39m�[2m).�[22mtoHaveScreenshot�[2m(�[22m�[32mexpected�[39m�[2m)�[22m
...

1490:  ────────────────────────────────────────────────────────────────────────────────────────────────
1491:  attachment #2: top-1-actual.png (image/png) ────────────────────────────────────────────────────
1492:  test-results/vrt-vrt-top-Mobile-Safari-retry2/top-1-actual.png
1493:  ────────────────────────────────────────────────────────────────────────────────────────────────
1494:  attachment #3: top-1-diff.png (image/png) ──────────────────────────────────────────────────────
1495:  test-results/vrt-vrt-top-Mobile-Safari-retry2/top-1-diff.png
1496:  ────────────────────────────────────────────────────────────────────────────────────────────────
1497:  2) [Mobile Safari] › tests/e2e/page.test.ts:25:5 › Table node should be highlighted when clicked ─
1498:  Error: �[2mexpect(�[22m�[31mreceived�[39m�[2m).�[22mtoBe�[2m(�[22m�[32mexpected�[39m�[2m) // Object.is equality�[22m
...

1503:  > 39 |   expect(firstChild).toBe('table-node-highlighted')
1504:  |                      ^
1505:  40 | })
1506:  41 |
1507:  42 | test('Edge animation should be triggered when table node is clicked', async ({
1508:  at /home/runner/work/liam/liam/frontend/packages/e2e/tests/e2e/page.test.ts:39:22
1509:  Slow test file: [Mobile Safari] › tests/e2e/toolbar.test.ts (56.4s)
1510:  Consider running tests from slow files in parallel, see https://playwright.dev/docs/test-parallel.
1511:  1 failed
1512:  [Mobile Safari] › tests/vrt/vrt.test.ts:21:5 › top ─────────────────────────────────────────────
1513:  1 flaky
1514:  [Mobile Safari] › tests/e2e/page.test.ts:25:5 › Table node should be highlighted when clicked ──
1515:  9 skipped
1516:  6 passed (2.3m)
1517:  ##[error]Process completed with exit code 1.

@MH4GF

This comment was marked as outdated.

@MH4GF MH4GF added this pull request to the merge queue Feb 17, 2025
Merged via the queue into main with commit 970305d Feb 17, 2025
19 checks passed
@MH4GF MH4GF deleted the changeset-release/main branch February 17, 2025 04:36
@MH4GF
Copy link
Member

MH4GF commented Feb 18, 2025

/giselle generate-release-notes

Highlights

  • Pre-calculate layout for Related Tables before displaying in Main Area
    • Tidy up of related tables is no longer necessary.

Releases

@liam-hq/[email protected]

Minor Changes

  • #739 - ✨ Pre-calculate layout for Related Tables before displaying in Main Area / Thanks @junkisai!

Patch Changes

  • #708 - ♻️ Swap useReactflow's fitView() with a custom fitView() function / Thanks @junkisai!
  • #708 - ♻️ Remove useSyncHiddenNodesChange hook and update VisibilityButton to manage node visibility directly / Thanks @junkisai!
  • #708 - ♻️ Consolidate table selection/deselection logic into useTableSelection hook / Thanks @junkisai!
  • #708 - 🐛 Fix RelatedTables additional buttons prop rendering / Thanks @junkisai!
  • #708 - ♻️ Limit useAutoLayout functionality to layout calculations only / Thanks @junkisai!
  • #726 - feat: expose all useReactflow functions in useCustomReactflow and replace direct useReactflow usages / Thanks @devin-ai-integration!
  • #739 - ✨ Pre-calculate layout for Related Tables before displaying in Main Area / Thanks @junkisai!
  • #708 - ♻️ Swap useReactflow's fitView() with a custom fitView() function / Thanks @junkisai!
  • #708 - ♻️ Remove useSyncHiddenNodesChange hook and update VisibilityButton to manage node visibility directly / Thanks @junkisai!
  • #708 - ♻️ Consolidate table selection/deselection logic into useTableSelection hook / Thanks @junkisai!
  • #708 - 🐛 Fix RelatedTables additional buttons prop rendering / Thanks @junkisai!
  • #708 - ♻️ Limit useAutoLayout functionality to layout calculations only / Thanks @junkisai!

Copy link

giselles-ai bot commented Feb 18, 2025

🚀 Release Notes for @liam-hq/[email protected]

🎉 Highlights

✨ Pre-calculate Layout for Related Tables

Say goodbye to the manual tidy-up of related tables! Now, layouts are pre-calculated before displaying them in the main area, ensuring a seamless and organized experience.

🔄 Releases

Minor Changes

  • #739 - ✨ Pre-calculate layout for Related Tables before displaying in Main Area / Thanks @junkisai!

Patch Changes

  • #708 - ♻️ Swap useReactflow's fitView() with a custom fitView() function / Thanks @junkisai!
  • #708 - ♻️ Remove useSyncHiddenNodesChange hook and update VisibilityButton to manage node visibility directly / Thanks @junkisai!
  • #708 - ♻️ Consolidate table selection/deselection logic into useTableSelection hook / Thanks @junkisai!
  • #708 - 🐛 Fix RelatedTables additional buttons prop rendering / Thanks @junkisai!
  • #708 - ♻️ Limit useAutoLayout functionality to layout calculations only / Thanks @junkisai!
  • #726 - feat: expose all useReactflow functions in useCustomReactflow and replace direct useReactflow usages / Thanks @devin-ai-integration!

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.

1 participant