-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
chore(main): release 0.0.231 #2434
chore(main): release 0.0.231 #2434
Conversation
9aa6637
to
80db30a
Compare
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me!
- Reviewed the entire pull request up to 80db30a
- Looked at
19
lines of code in1
files - Took 43 seconds to review
More info
- Skipped
0
files when reviewing. - Skipped posting
1
additional comments because they didn't meet confidence threshold of50%
.
1. CHANGELOG.md:1
:
- Assessed confidence :
0%
- Comment:
The changelog update looks correct, reflecting the new version and the changes made. Good job on keeping the changelog up to date. - Reasoning:
This PR is a release PR, which is automatically generated. The changes are only in the CHANGELOG.md file, reflecting the new version and the changes made in the new version. There is no actual code change in this PR, so there's no code to review for bugs or best practices. The only thing to check is whether the changelog correctly reflects the changes made in the new version.
Workflow ID: wflow_z1GSo3X96khF0Hv7
Not what you expected? You can customize the content of the reviews using rules. Learn more here.
⌛ You have 2 days remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev
80db30a
to
26235bd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me!
- Performed an incremental review on 26235bd
- Looked at
19
lines of code in1
files - Took 2 minutes and 4 seconds to review
More info
- Skipped
0
files when reviewing. - Skipped posting
1
additional comments because they didn't meet confidence threshold of50%
.
1. CHANGELOG.md:1
:
- Assessed confidence :
100%
- Grade:
0%
- Comment:
The index before and after the change is the same, which is not possible if there were changes made to the file. Please check the diff generation process. - Reasoning:
This is a release PR, and it seems to be auto-generated. The changes are only in the CHANGELOG.md file, and they seem to be correctly reflecting the changes made in the mentioned PRs. However, there seems to be a mistake in the diff. The index before and after the change is the same, which is not possible if there were changes made to the file.
Workflow ID: wflow_P41iBoqhAvRus12o
Not what you expected? You can customize the content of the reviews using rules. Learn more here.
⌛ You have 2 days remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev
26235bd
to
8790166
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me!
- Performed an incremental review on 8790166
- Looked at
20
lines of code in1
files - Took 1 minute and 4 seconds to review
More info
- Skipped
0
files when reviewing. - Skipped posting
1
additional comments because they didn't meet confidence threshold of50%
.
1. CHANGELOG.md:1
:
- Assessed confidence :
0%
- Comment:
The changelog update seems to be in order, with the correct version number and date. The included changes and their respective PR links are also correctly formatted and informative. - Reasoning:
This is a release PR, and it seems to be auto-generated. The changes in the PR are only in the CHANGELOG.md file, reflecting the new version and the features added. There doesn't seem to be any code changes in this PR, so there's no code to review for bugs or best practices. However, I'll check the formatting and information in the CHANGELOG.md file.
Workflow ID: wflow_IiropksMQGDdoGiM
Not what you expected? You can customize the content of the reviews using rules. Learn more here.
⌛ You have 2 days remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev
8790166
to
91756da
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me!
- Performed an incremental review on 91756da
- Looked at
21
lines of code in1
files - Took 1 minute and 39 seconds to review
More info
- Skipped
0
files when reviewing. - Skipped posting
1
additional comments because they didn't meet confidence threshold of50%
.
1. CHANGELOG.md:9
:
- Assessed confidence :
33%
- Comment:
There's a small typo in the description of the change made in PR 2437. It should be 'rather than python loop', not 'rather that python loop'. - Reasoning:
This PR is a release PR, and it's updating the CHANGELOG.md file. The changes seem to be in order, with the new version number and the date correctly updated. The list of changes also seems to be in order, with the correct PR links and descriptions. However, there's a typo in the description of one of the changes. It says 'rather that python loop' instead of 'rather than python loop'. This is a minor issue, but it's worth pointing out for the sake of clarity and professionalism in the documentation.
Workflow ID: wflow_YwQGibdFrn0Xi4kH
Not what you expected? You can customize the content of the reviews using rules. Learn more here.
⌛ You have 2 days remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev
91756da
to
8f584e6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me!
- Performed an incremental review on 8f584e6
- Looked at
22
lines of code in1
files - Took 1 minute and 32 seconds to review
More info
- Skipped
0
files when reviewing. - Skipped posting
1
additional comments because they didn't meet confidence threshold of50%
.
1. CHANGELOG.md:1
:
- Assessed confidence :
0%
- Comment:
The changes in the CHANGELOG.md file are correctly formatted and informative. Good job! - Reasoning:
This is a release PR, and it seems to be auto-generated. The changes are only in the CHANGELOG.md file, and they seem to be correctly formatted and informative. There's no code to review here, so there's no chance of bugs, performance issues, or security issues. The PR description is also clear and informative.
Workflow ID: wflow_I46J4EggbPUABmoV
Not what you expected? You can customize the content of the reviews using rules. Learn more here.
⌛ You have 2 days remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev
8f584e6
to
8dfdee0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me!
- Performed an incremental review on 8dfdee0
- Looked at
23
lines of code in1
files - Took 1 minute and 53 seconds to review
More info
- Skipped
0
files when reviewing. - Skipped posting
1
additional comments because they didn't meet confidence threshold of50%
.
1. CHANGELOG.md:8
:
- Assessed confidence :
100%
- Grade:
0%
- Comment:
There's a typo in the description of the change made in PR 2437. It should be 'rather than python loop', not 'rather that python loop'. - Reasoning:
The PR is a release PR, and it's updating the CHANGELOG.md file. The changes seem to be in order, with the new version number and the date correctly updated. The list of changes also seems to be in order, with the correct PR links and descriptions. However, there's a typo in one of the descriptions: 'rather that python loop' should be 'rather than python loop'.
Workflow ID: wflow_2sCUAZVN9Px41ixx
Not what you expected? You can customize the content of the reviews using rules. Learn more here.
⌛ You have 1 day remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev
8dfdee0
to
4a56f58
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me!
- Performed an incremental review on 4a56f58
- Looked at
24
lines of code in1
files - Took 2 minutes and 32 seconds to review
More info
- Skipped
0
files when reviewing. - Skipped posting
1
additional comments because they didn't meet confidence threshold of50%
.
1. CHANGELOG.md:1
:
- Assessed confidence :
100%
- Grade:
0%
- Comment:
The index of the file before and after the changes is the same. This could indicate that the file hasn't actually been changed. Please double-check this. - Reasoning:
This PR is a release PR, and it's updating the CHANGELOG.md file. The changes seem to be in order, with the new version number and the list of changes. However, the diff shows that the index of the file before and after the changes is the same, which is unusual. This could be a mistake in the diff generation, or it could mean that the file hasn't actually been changed. I need to comment on this to bring it to the attention of the PR author.
Workflow ID: wflow_hzfSlV3sgz64nPtI
Not what you expected? You can customize the content of the reviews using rules. Learn more here.
⌛ You have 1 day remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev
4a56f58
to
897976b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me!
- Performed an incremental review on 897976b
- Looked at
25
lines of code in1
files - Took 1 minute and 18 seconds to review
More info
- Skipped
0
files when reviewing. - Skipped posting
1
additional comments because they didn't meet confidence threshold of50%
.
1. CHANGELOG.md:1
:
- Assessed confidence :
50%
- Comment:
The index before and after the change is the same. This could be a mistake or a bug in the tool that generated the PR. Please double-check. - Reasoning:
This PR is a release PR, and it's updating the CHANGELOG.md file. The changes seem to be in order, with the new version number and the list of changes. However, the diff shows that the index before and after the change is the same, which is unusual. This could be a mistake or a bug in the tool that generated the PR.
Workflow ID: wflow_7zLMbd1cRaT4Y4o4
Not what you expected? You can customize the content of the reviews using rules. Learn more here.
⌛ You have 1 day remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev
897976b
to
da2e26e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me!
- Performed an incremental review on da2e26e
- Looked at
26
lines of code in1
files - Took 1 minute and 44 seconds to review
More info
- Skipped
0
files when reviewing. - Skipped posting
1
additional comments because they didn't meet confidence threshold of50%
.
1. CHANGELOG.md:1
:
- Assessed confidence :
0%
- Comment:
The changes in the CHANGELOG.md file look good. They are well-formatted and include the correct information about the changes in the new version. - Reasoning:
This PR is a release PR, which is mostly about updating the CHANGELOG.md file. The diff shows that the changes are only in the CHANGELOG.md file, and they seem to be correctly formatted and include the correct information about the changes in the new version. There are no code changes to review, so there's no need to check for bugs or violations of best practices. The PR description is also well-written and includes all the necessary information.
Workflow ID: wflow_5AD7yhIxzrpCmGuj
Not what you expected? You can customize the content of the reviews using rules. Learn more here.
⌛ You have 1 day remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev
da2e26e
to
299d978
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❌ Changes requested.
- Performed an incremental review on 299d978
- Looked at
27
lines of code in1
files - Took 1 minute and 50 seconds to review
More info
- Skipped
0
files when reviewing. - Skipped posting
0
additional comments because they didn't meet confidence threshold of50%
.
Workflow ID: wflow_KDmg0HRLxxcnCkIe
Want Ellipsis to fix these issues? Tag @ellipsis-dev
in a comment. We'll respond in a few minutes. Learn more here.
⌛ You have 23 hours remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev
## What's Changed | ||
* feat(assistants): Add user usage update and pricing calculation to ITO assistant by @StanGirard in https://github.com/QuivrHQ/quivr/pull/2433 | ||
* feat(assistant): improve prompt summary by @StanGirard in https://github.com/QuivrHQ/quivr/pull/2435 | ||
* feat(assistants): Add PDF generation functionality and nice emails by @StanGirard in https://github.com/QuivrHQ/quivr/pull/2436 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's a small typo in the changelog entry. It should be 'rather than' instead of 'rather that'.
* feat(assistants): Add PDF generation functionality and nice emails by @StanGirard in https://github.com/QuivrHQ/quivr/pull/2436 | |
* feat(analytics): rely on sql rather than python loop for brains by @StanGirard in https://github.com/QuivrHQ/quivr/pull/2437 |
299d978
to
9ebbcc3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me!
- Performed an incremental review on 9ebbcc3
- Looked at
29
lines of code in1
files - Took 7 minutes and 34 seconds to review
More info
- Skipped
0
files when reviewing. - Skipped posting
1
additional comments because they didn't meet confidence threshold of50%
.
1. CHANGELOG.md:1
:
- Assessed confidence :
0%
- Comment:
This PR seems to be auto-generated for a new release. It only includes changes to the CHANGELOG.md file, and the PR description provides a clear summary of the changes included in this release. There are no code changes to review in this PR. - Reasoning:
This is a release PR, and it seems to be auto-generated. The changes in the PR are only in the CHANGELOG.md file, reflecting the new version and the changes made. There doesn't seem to be any code changes in this PR, so there's no code to review for bugs or best practices. The PR description is clear and provides links to all the changes included in this release.
Workflow ID: wflow_TNxFGoO1Cb9LSB1R
Not what you expected? You can customize the content of the reviews using rules. Learn more here.
⌛ You have 18 hours remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev
9ebbcc3
to
5c421ee
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me!
- Performed an incremental review on 5c421ee
- Looked at
30
lines of code in1
files - Took 4 minutes and 4 seconds to review
More info
- Skipped
0
files when reviewing. - Skipped posting
1
additional comments because they didn't meet confidence threshold of50%
.
1. CHANGELOG.md:1
:
- Assessed confidence :
0%
- Comment:
The version number seems to be incremented correctly, and the changes listed match the changes made in the commits. No issues found. - Reasoning:
This is a release PR, and it seems to be auto-generated. The changes in the PR are only in the CHANGELOG.md file, reflecting the new version and the changes made. There is no actual code change in this PR, so there's no code to review for bugs or violations. However, I'll check if the version number is incremented correctly and if the changes listed match the changes made in the commits.
Workflow ID: wflow_lM7OlqeTgDRfTKBV
Not what you expected? You can customize the content of the reviews using rules. Learn more here.
⌛ You have 17 hours remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev
5c421ee
to
a16fa4d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me!
- Performed an incremental review on a16fa4d
- Looked at
31
lines of code in1
files - Took 1 minute and 48 seconds to review
More info
- Skipped
0
files when reviewing. - Skipped posting
1
additional comments because they didn't meet confidence threshold of50%
.
1. CHANGELOG.md:1
:
- Assessed confidence :
0%
- Comment:
The changes in the CHANGELOG.md file correctly reflect the updates made in this version. No issues found. - Reasoning:
This is a release PR, and the changes are only in the CHANGELOG.md file. The changes seem to be correctly reflecting the updates made in the version. There are no code changes to review here.
Workflow ID: wflow_j7mwwvN8n5T4WSUO
Not what you expected? You can customize the content of the reviews using rules. Learn more here.
⌛ You have 7 hours remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev
🤖 Release is at https://github.com/QuivrHQ/quivr/releases/tag/v0.0.231 🌻 |
🤖 I have created a release beep boop
0.0.231 (2024-04-19)
What's Changed
Full Changelog: v0.0.230...v0.0.231
This PR was generated with Release Please. See documentation.
Summary:
This PR releases version 0.0.231, introducing several feature enhancements and bug fixes across the assistant, analytics, Docker, and frontend modules.
Key points:
Generated with ❤️ by ellipsis.dev