Skip to content

Conversation

@aseembits93
Copy link
Contributor

@aseembits93 aseembits93 commented May 19, 2025

User description

Tests will pass now as tiktoken is not used anywhere in the code


PR Type

Other


Description

  • Remove unused tiktoken dependency from project

Changes walkthrough 📝

Relevant files
Dependencies
pyproject.toml
Remove tiktoken dependency                                                             

pyproject.toml

  • Deleted tiktoken dependency line
+0/-1     

Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • @github-actions
    Copy link

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Dependency Removal

    Ensure that all references to tiktoken (imports, usages, documentation) are removed or updated to prevent build or runtime errors.

    pytest = ">=7.0.0,!=8.3.4"
    gitpython = ">=3.1.31"
    libcst = ">=1.0.1"
    jedi = ">=0.19.1"
    timeout-decorator = ">=0.5.0"
    pytest-timeout = ">=2.1.0"
    tomlkit = ">=0.11.7"
    unittest-xml-reporting = ">=3.2.0"

    @github-actions
    Copy link

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @aseembits93 aseembits93 deleted the remove-tiktoken-pyproject branch May 28, 2025 02:27
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    2 participants