Skip to content

Conversation

bagel897
Copy link
Contributor

Motivation

  • Helpful for LSP support

Content

  • Move all file I/O into it's own class

Copy link

codecov bot commented Feb 10, 2025

Codecov Report

Attention: Patch coverage is 98.49624% with 2 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/codegen/sdk/core/file.py 84.61% 2 Missing ⚠️
Additional details and impacted files

@bagel897 bagel897 added the parse-tests Enable parse tests for this PR label Feb 11, 2025
@bagel897 bagel897 marked this pull request as ready for review February 11, 2025 00:56
@bagel897 bagel897 requested review from codegen-team and a team as code owners February 11, 2025 00:56
@bagel897 bagel897 enabled auto-merge (squash) February 11, 2025 00:56
@bagel897 bagel897 merged commit acf2ea4 into develop Feb 11, 2025
26 of 27 checks passed
@bagel897 bagel897 deleted the eagarwal-cg-10754-split-out-file-io branch February 11, 2025 01:10
Copy link
Contributor

🎉 This PR is included in version 0.7.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
parse-tests Enable parse tests for this PR released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants