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.

Sign up for GitHub

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

Jump to bottom

[DRAFT] update_file with difflib #4079

Closed
wants to merge 13 commits into from
Closed

[DRAFT] update_file with difflib #4079

wants to merge 13 commits into from

Conversation

Boostrix
Copy link
Contributor

#3643 (update_file with diff support):

difflib-agpt

Background

Changes

Documentation

Test Plan

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

bfalans and others added 13 commits May 1, 2023 23:24
@bfalans
update_file method to edit files
b89fa0c
@bfalans
Merge branch 'master' into update_file
8530a19
@bfalans
rewritten update_file method using re and able to change one or all o…
58bbbde 
…ccurences of specified text
@bfalans
Merge https://github.com/bfalans/revised_file_operations into update_…
0a07dcf 
…file
@bfalans
Merge branch 'Significant-Gravitas:master' into update_file
ccaaa54
@bfalans
Merge branch 'update_file' of https://github.com/bfalans/revised_file…
e7510b9 
…_operations into update_file
@bfalans
fixed a typo in the code
fd02dec
@bfalans
Merge branch 'master' into update_file
16ba0c1
@bfalans
Merge branch 'master' into update_file
7979e7a
@bfalans
Merge branch 'master' into update_file
942163b
@bfalans
Merge branch 'master' into update_file
0fbefc2
@bfalans
Merge branch 'master' into update_file
e87c25c
@Boostrix
based on talks here: Significant-Gravitas#3643 (comment)
516a629
@vercel
Copy link

vercel bot commented May 10, 2023 •
edited
Loading

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

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
docs ⬜️ Ignored (Inspect) May 10, 2023 1:00pm

@Boostrix Boostrix mentioned this pull request May 10, 2023
Closed
5 tasks
@Boostrix Boostrix mentioned this pull request May 11, 2023
Closed
1 task
@ntindle
Copy link
Member

ntindle commented May 18, 2023

Why draft?

@Boostrix
Copy link
Contributor Author

Boostrix commented May 18, 2023 •
edited
Loading

Depended on something under review and would be up to people if it's useful as is or not - personally just found it easier to visualize changes that way.

Could / should probably make this a config option so that this can be enabled / disabled like people prefer

@Boostrix Boostrix marked this pull request as ready for review May 18, 2023 17:15
@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label May 25, 2023
@github-actions
Copy link
Contributor

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

@Pwuts Pwuts added the fridge Items that can't be processed right now but can be of use or inspiration later label Sep 8, 2023
@Pwuts
Copy link
Member

Pwuts commented Sep 8, 2023

Not applicable at this time, but possibly interesting in the future. Labeled with fridge and closing for now. :)

@Pwuts Pwuts closed this Sep 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
conflicts Automatically applied to PRs with merge conflicts fridge Items that can't be processed right now but can be of use or inspiration later size/l
Projects
AutoGPT development kanban
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Boostrix @ntindle @Pwuts @bfalans