Skip to content

Conversation

@caroljung-cg
Copy link
Contributor

Motivation

Content

Testing

Please check the following before marking your PR as ready for review

  • I have added tests for my changes
  • I have updated the documentation or added new documentation as needed

@caroljung-cg caroljung-cg marked this pull request as ready for review February 7, 2025 01:38
@caroljung-cg caroljung-cg requested review from a team and codegen-team as code owners February 7, 2025 01:38
@caroljung-cg caroljung-cg requested review from christinewangcw and removed request for a team February 7, 2025 01:38
@caroljung-cg caroljung-cg enabled auto-merge (squash) February 7, 2025 01:38
@codecov
Copy link

codecov bot commented Feb 7, 2025

Codecov Report

Attention: Patch coverage is 42.85714% with 4 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/codegen/git/clients/git_repo_client.py 50.00% 2 Missing ⚠️
src/codegen/git/clients/github_client.py 50.00% 1 Missing ⚠️
.../codegen/git/repo_operator/remote_repo_operator.py 0.00% 1 Missing ⚠️
Additional details and impacted files

@caroljung-cg caroljung-cg merged commit 1d77a05 into develop Feb 7, 2025
23 of 24 checks passed
@caroljung-cg caroljung-cg deleted the git-migration-fixes branch February 7, 2025 01:44
jayhack pushed a commit that referenced this pull request Feb 9, 2025
tkfoss pushed a commit that referenced this pull request Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants