Skip to content

Conversation

@drQedwards
Copy link

@drQedwards drQedwards commented Jul 29, 2025

I have read the CLA Document and I hereby sign the CLA

Summary

Brief description of the changes in this PR.

Type of Change

  • Bug fix
  • New feature
  • Performance improvement
  • Documentation/Tests

Objective

For new features and performance improvements: Clearly describe the objective and rationale for this change.

Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • All existing tests pass

Breaking Changes

  • This PR contains breaking changes

If this is a breaking change, describe:

  • What functionality is affected
  • Migration path for existing users

Checklist

  • Code follows project style guidelines (make lint passes)
  • Self-review completed
  • Documentation updated where necessary
  • No secrets or sensitive information committed

Related Issues

Closes #[issue number]


Important

Adds a signature section to Zep-CLA.md indicating CLA agreement by @drQedwards.

  • Documentation:
    • Adds a signature section to Zep-CLA.md with the statement "I have read the CLA Document and I hereby sign the CLA @drQedwards".

This description was created by Ellipsis for b1033f2. You can customize this summary. It will automatically update as commits are pushed.

I have read the CLA Document and I hereby sign the CLA
@danielchalef
Copy link
Member


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@drQedwards
Copy link
Author

Please see PR #784 this is to sign the CLA FOR THAT PR!

sorry I’m not used to doing signing a CLA this way
image

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Important

Looks good to me! 👍

Reviewed everything up to b1033f2 in 43 seconds. Click for details.
  • Reviewed 12 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 2 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. Zep-CLA.md:43
  • Draft comment:
    Consider updating the header to use Title Case (e.g., '## Signature for CLA') to maintain consistency with the rest of the document.
  • Reason this comment was not posted:
    Confidence changes required: 80% <= threshold 80% None
2. Zep-CLA.md:45
  • Draft comment:
    Consider including additional signature details such as the signer's full legal name and a date for clarity in this legal document.
  • Reason this comment was not posted:
    Confidence changes required: 80% <= threshold 80% None

Workflow ID: wflow_obqBJsw5rM8bjGGC

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@drQedwards drQedwards mentioned this pull request Jul 29, 2025
12 tasks
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