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: address api_request issues on Python 3.11+ #5643

Merged
merged 3 commits into from
Jan 13, 2025

Conversation

mieslep
Copy link
Contributor

@mieslep mieslep commented Jan 13, 2025

Addresses nightly build failures seen on 3.11 and 3.12 versions.

@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. bug Something isn't working labels Jan 13, 2025
@github-actions github-actions bot added bug Something isn't working and removed bug Something isn't working labels Jan 13, 2025
- Enhanced assertion for metadata keys to include raw result in error message for better debugging.
- Updated file reading method to use aiofiles for consistency with asynchronous file operations.
- Ensured that the test validates the existence and content of saved files accurately.
- Replaced aiofile with aiofiles for consistent asynchronous file operations.
- Added logic to ensure parent directories are created before saving files.
- Improved file existence check by using exclusive creation mode to handle filename conflicts.
- Updated file writing methods to handle both binary and text content appropriately.
@github-actions github-actions bot added bug Something isn't working and removed bug Something isn't working labels Jan 13, 2025
@ogabrielluiz ogabrielluiz merged commit f85d447 into langflow-ai:main Jan 13, 2025
12 of 14 checks passed
mieslep added a commit to mieslep/langflow that referenced this pull request Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants