Skip to content

chore: add test for cjs #110

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 8 commits into from
Jun 3, 2025
Merged

chore: add test for cjs #110

merged 8 commits into from
Jun 3, 2025

Conversation

marcusschiesser
Copy link
Contributor

No description provided.

@changeset-bot changeset-bot
Copy link

changeset-bot bot commented May 28, 2025 •
edited
Loading

⚠️ No Changeset found

Latest commit: eb9b242

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR
@pkg-pr-new pkg.pr.new
Copy link

pkg-pr-new bot commented May 28, 2025 •
edited
Loading

Open in StackBlitz

npm i https://pkg.pr.new/run-llama/llama-flow/@llama-flow/core@110

npm i https://pkg.pr.new/run-llama/llama-flow/@llama-flow/http@110

npm i https://pkg.pr.new/run-llama/llama-flow/@llama-flow/llamaindex@110

commit: eb9b242
@marcusschiesser marcusschiesser mentioned this pull request May 28, 2025
Closed
14 tasks
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented May 28, 2025 •
edited
Loading

Deployed at https://llamaindex-ts-7m69nq1r2.pr.staging.llamaindex.ai
CdTgr
CdTgr suggested changes May 28, 2025
View reviewed changes
Copy link
Contributor

@CdTgr CdTgr left a comment

Choose a reason for hiding this comment

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

Apply these changes and the test cases will fail.
@CdTgr CdTgr mentioned this pull request May 28, 2025
Merged
2 tasks
marcusschiesser and others added 2 commits May 29, 2025 10:29
@marcusschiesser @CdTgr
Update tests/cjs/workflow-cjs.test.ts
b017ecf 
Co-authored-by: GhosT <adarsmohan76@icloud.com>
@marcusschiesser @CdTgr
Update tests/cjs/workflow-cjs.test.ts
1378460 
Co-authored-by: GhosT <adarsmohan76@icloud.com>
@CdTgr CdTgr mentioned this pull request Jun 2, 2025
Merged
@CdTgr
Copy link
Contributor

CdTgr commented Jun 2, 2025

@marcusschiesser I assume this PR requires the latest changes in the main branch for the tests to work.
CdTgr and others added 5 commits June 3, 2025 12:12
@CdTgr @himself65 @marcusschiesser
fix: build and export all middlewares to esm and cjs modules (#111)
658db65 
Co-authored-by: Alex Yang <himself65@outlook.com>
@github-actions @himself65 @marcusschiesser
Version Packages (#112)
c4f61c5 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: himself65 <14026360+himself65@users.noreply.github.com>
@marcusschiesser
Add debugging to CJS test to understand CI module resolution issue
aedfb71
@marcusschiesser
Fix CI: ensure packages are built before running tests
f20380c
@marcusschiesser
Clean up debugging output from CJS test
eb9b242
@himself65 himself65 merged commit 2865b31 into main Jun 3, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants
@marcusschiesser @CdTgr @himself65