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

Revert "Filtering out ANSI escape codes in printed assistant thoughts" #4860

Closed

Conversation

lc0rp
Copy link
Contributor

@lc0rp lc0rp commented Jul 1, 2023

Reverts #4812 and re-applies only the intended security fixes

This reverts commit ebe1861.

(cherry picked from commit e9ed0a48e72d96289b0b217424b85a01140399e4)

lc0rp and others added 2 commits July 1, 2023 07:38
…Significant-Gravitas#4812)"

This reverts commit ebe1861.

(cherry picked from commit e9ed0a48e72d96289b0b217424b85a01140399e4)
…icant-Gravitas#4810)

Co-authored-by: merwanehamadi <merwanehamadi@gmail.com>
Co-authored-by: Luke K (pr-0f3t) <2609441+lc0rp@users.noreply.github.com>
(cherry picked from commit bafcdce)
(cherry picked from commit b69816823c34ac4c2ef141dc8ec195f55636eff6)
@github-actions
Copy link
Contributor

github-actions bot commented Jul 1, 2023

This PR exceeds the recommended size of 500 lines. Please make sure you are NOT addressing multiple issues with one PR.

@codecov
Copy link

codecov bot commented Jul 1, 2023

Codecov Report

Patch coverage: 73.45% and project coverage change: -0.20 ⚠️

Comparison is base (80151dd) 70.81% compared to head (9ba25db) 70.62%.

Additional details and impacted files
@@                Coverage Diff                 @@
##           release-v0.4.4    #4860      +/-   ##
==================================================
- Coverage           70.81%   70.62%   -0.20%     
==================================================
  Files                  69       68       -1     
  Lines                3358     3336      -22     
  Branches              545      535      -10     
==================================================
- Hits                 2378     2356      -22     
- Misses                802      803       +1     
+ Partials              178      177       -1     
Impacted Files Coverage Δ
autogpt/plugins/__init__.py 73.54% <ø> (ø)
autogpt/main.py 21.05% <33.33%> (ø)
autogpt/configurator.py 30.37% <69.23%> (ø)
autogpt/config/config.py 68.63% <73.75%> (-5.31%) ⬇️
autogpt/llm/providers/openai.py 90.74% <100.00%> (-0.09%) ⬇️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Jul 7, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jul 7, 2023

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

@collijk
Copy link
Contributor

collijk commented Jul 7, 2023

Can you provide some more background on what's going on here? What was wrong with the original PR?

@Pwuts Pwuts closed this Jul 11, 2023
@Pwuts
Copy link
Member

Pwuts commented Jul 11, 2023

@collijk the intent was to get a specific fix into the release branch, but because the patch branch was based on master, the merge exceeded the scope of the PR: #4812

@lc0rp lc0rp deleted the release-v0.4.4 branch July 23, 2023 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conflicts Automatically applied to PRs with merge conflicts size/xl
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants