-
Notifications
You must be signed in to change notification settings - Fork 44.6k
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
Add file op tests #2205
Add file op tests #2205
Conversation
CI is red due to the use of is_relative_to here https://github.com/Significant-Gravitas/Auto-GPT/blob/5802f17726e69d4a764254c4f3d6a259aa410839/autogpt/workspace.py#L38, which only exists in 3.9+ unsure how to resolve this |
|
I tried that before, but it was rejected as someone didn't like the fact that I was updating to an older method call. Let's try again |
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request. |
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly. |
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request. |
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly. |
We resolved it by requiring Python 3.10+. Pathlib unfortunately wasn't the only one breaking on older Python versions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot! A few tweak requests below.
Background
Added file operations tests, to combat the increasing number of modifications to this file which break file io
Changes
Add and test file ops
Documentation
Lightly documented tests, but mostly self documenting - feel free to call for revisions if necessary
Test Plan
Ran new unit tests
PR Quality Checklist