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

feat: release v0.6.17 #1221

Merged
merged 3 commits into from
Jan 24, 2025
Merged

feat: release v0.6.17 #1221

merged 3 commits into from
Jan 24, 2025

Conversation

angrybayblade
Copy link
Collaborator

@angrybayblade angrybayblade commented Jan 24, 2025

Important

Update Composio to v0.6.17, adding new actions, apps, tags, and updating dependencies.

  • Version Update:
    • Update version to 0.6.17 in __version__.py, Dockerfile, and all setup.py files.
  • Actions and Apps:
    • Add new Action entries for APALEO, BINANCE, COINBASE, COMPOSIO_SEARCH, DROPBOX, FIREFLIES, LINKUP, ONEPAGE, OPEN_SEA in action.pyi.
    • Add new App entries for APALEO, BINANCE, COINBASE, COMPOSIO_SEARCH, FIREFLIES, LINKUP, ONEPAGE, OPEN_SEA in app.pyi.
    • Add new Tag entries for APALEO, BINANCE, MEM0, ONEPAGE, OPEN_SEA in tag.pyi.
  • Dependencies:
    • Add pyyaml>=6.0.2 to setup.py for composio_core.
    • Update composio-core version in Dockerfile to 0.6.17.

This description was created by Ellipsis for 51551fd. It will automatically update as commits are pushed.

Copy link

vercel bot commented Jan 24, 2025

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

Name Status Preview Comments Updated (UTC)
composio ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 24, 2025 6:25am

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.

👍 Looks good to me! Reviewed everything up to f2836fb in 12 seconds

More details
  • Looked at 241 lines of code in 19 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. python/dockerfiles/Dockerfile:19
  • Draft comment:
    Using export in a RUN command does not persist the environment variable in subsequent RUN commands. Consider using ENV to set the PATH variable persistently.
  • Reason this comment was not posted:
    Confidence changes required: 50%
    The PR is primarily about version updates across multiple files. The changes are consistent and seem to be correctly applied. However, there is a minor issue in the Dockerfile regarding the use of the export command in a RUN instruction, which does not persist the environment variable.

Workflow ID: wflow_g33KwHvnINoo52mG


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@shreysingla11
Copy link
Collaborator

Code Review Summary

I've reviewed the version bump changes in PR #1221. Here's my assessment:

Changes Overview

  • Core package version bump from 0.6.16 to 0.6.17
  • All plugins synchronized to version 0.6.17
  • Docker image updated to use new version
  • SWE package version bump from 0.3.17 to 0.3.18

Quality Assessment

✅ Version numbers are consistently updated across all components
✅ Changes follow semantic versioning pattern
✅ All necessary files are included in the update
✅ No inconsistencies or typos found in version numbers

The changes look clean and well-organized. This is a standard version bump PR that maintains consistency across all package components. No issues found.

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.

👍 Looks good to me! Incremental review on 443e2d9 in 8 seconds

More details
  • Looked at 12 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. python/setup.py:55
  • Draft comment:
    Ensure that adding 'pyyaml>=6.0.2' does not conflict with other dependencies or cause compatibility issues.
  • Reason this comment was not posted:
    Confidence changes required: 50%
    The addition of 'pyyaml>=6.0.2' in the core_requirements list is consistent with the PR title indicating a version release. However, it's important to ensure that this addition does not conflict with other dependencies or cause any compatibility issues.

Workflow ID: wflow_Zuack6khuclwj0HP


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

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.

❌ Changes requested. Incremental review on 51551fd in 39 seconds

More details
  • Looked at 365 lines of code in 3 files
  • Skipped 0 files when reviewing.
  • Skipped posting 0 drafted comments based on config settings.

Workflow ID: wflow_jrzVUuYfj1D66pYh


Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

return version
version = "_".join(version.split(".")).lstrip("v")
if version.count("_") != 1:
raise InvalidVersionString(version)
Copy link
Contributor

Choose a reason for hiding this comment

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

Consider providing a more descriptive error message when raising InvalidVersionString to give more context about the error.

@angrybayblade angrybayblade merged commit de68cff into master Jan 24, 2025
21 of 25 checks passed
@angrybayblade angrybayblade deleted the release/v0.6.17 branch January 24, 2025 06:37
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