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

test: Add integration test to install, build, run, and exit Eliza #79

Closed
wants to merge 112 commits into from

Conversation

jzvikart
Copy link
Collaborator

@jzvikart jzvikart commented Dec 6, 2024

@monilpat This is a working branch that we need only to enable workflow file. It's NOT working yet so it should NOT be merged upstream, but we need it in order to move on.

@monilpat monilpat changed the title Test/integration test poc test: Add integration test to install, build, run, and exit Eliza Dec 6, 2024
@monilpat
Copy link
Collaborator

monilpat commented Dec 6, 2024

@monilpat This is a working branch that we need only to enable workflow file. It's NOT working yet so it should NOT be merged upstream, but we need it in order to move on.

Cool sounds good thanks great work! Please let me know as soon as it is working and we can create a PR for upstream thanks!

@snobbee snobbee changed the base branch from main to sif-dev December 6, 2024 18:23
@jzvikart
Copy link
Collaborator Author

jzvikart commented Dec 7, 2024

@snobbee This PR has added a workflow which we need to test. It is not showing up any more, how can we trigger it?

@jzvikart jzvikart force-pushed the test/integration-test-poc branch from 221db9c to 6bccad4 Compare December 7, 2024 06:30
@jzvikart jzvikart changed the base branch from sif-dev to main December 9, 2024 12:01
@jzvikart jzvikart changed the base branch from main to sif-dev December 9, 2024 12:02
asianviking and others added 28 commits December 10, 2024 18:55
fix: revert llamacloud endpoint change
feat: add callback handler to runtime evaluate method
fix: docker trying to filter out missing docs package
feat: allow character.json settings models for open router
- NodePlugin is now only created when all required AWS credentials are present
  (AWS_ACCESS_KEY_ID, AWS_SECRET_ACCESS_KEY, AWS_REGION, AWS_S3_BUCKET)
- GoatPlugin is now only created when ALCHEMY_API_KEY is present
- Added type annotations for plugins to handle undefined cases

These changes prevent unnecessary plugin initialization when required
credentials are missing and improve type safety.

Issues introduced in: elizaOS#941 and
elizaOS#898
…my-and-aws-env

fix: Goat Plugin + AWS S3 Service error when env vars absent
@jzvikart jzvikart force-pushed the test/integration-test-poc branch from d918e05 to 107d9e9 Compare December 11, 2024 15:52
@jzvikart jzvikart closed this Dec 11, 2024
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.