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

fix the forgotten + symbol in parse_ability_result(...) in parser.py #5028

Merged
merged 3 commits into from
Jul 21, 2023

Conversation

cyrus-hawk
Copy link
Contributor

Background

The + symbol is forgotten in parse_ability_result(...) func inside parser.py.

Changes

Puts back the + symbol in its place.

Documentation

NA

Test Plan

NA

PR Quality Checklist

  • My pull request is atomic and focuses on a single change.
  • I have thoroughly tested my changes with multiple different prompts.
  • I have considered potential risks and mitigations for my changes.
  • I have documented my changes clearly and comprehensively.
  • I have not snuck in any "extra" small tweaks changes.
  • I have run the following commands against my code to ensure it passes our linters:
    black .
    isort .
    mypy
    autoflake --remove-all-unused-imports --recursive --ignore-init-module-imports --ignore-pass-after-docstring autogpt tests --in-place

@cyrus-hawk cyrus-hawk requested a review from collijk as a code owner July 21, 2023 06:49
@netlify
Copy link

netlify bot commented Jul 21, 2023

Deploy Preview for auto-gpt-docs canceled.

Name Link
🔨 Latest commit 3004814
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/64bad20de90d2d00088ffbbc

@collijk collijk self-assigned this Jul 21, 2023
@collijk collijk merged commit 12d1263 into Significant-Gravitas:master Jul 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants