Skip to content

Fix references for git related yaml fields#1526

Merged
deep1401 merged 4 commits intomainfrom
fix/git-references
Mar 12, 2026
Merged

Fix references for git related yaml fields#1526
deep1401 merged 4 commits intomainfrom
fix/git-references

Conversation

@deep1401
Copy link
Copy Markdown
Member

Solves the comment I mentioned in #1524

@sentry
Copy link
Copy Markdown

sentry bot commented Mar 11, 2026

Codecov Report

❌ Patch coverage is 14.28571% with 12 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
api/transformerlab/routers/experiment/task.py 0.00% 8 Missing ⚠️
api/transformerlab/routers/compute_provider.py 0.00% 4 Missing ⚠️

📢 Thoughts on this report? Let us know!

@paragon-review
Copy link
Copy Markdown

Paragon Summary

This pull request review analyzed 6 files and found no issues. The review examined code changes, potential bugs, security vulnerabilities, performance issues, and code quality concerns using automated analysis tools.

Paragon did not detect any problems in the current diff. Proceed with merge after your normal checks.

This PR fixes incorrect references for git-related YAML fields across the backend API, CLI, and frontend components, addressing a comment raised in PR #1524. The changes ensure consistent field naming for git configuration throughout the codebase.

Key changes:

  • Fixes references to git-related YAML fields across codebase
  • Updates API routers for compute providers and tasks
  • Modifies compute provider schema definitions
  • Updates CLI task command handling
  • Adjusts frontend Interactive and Tasks components

Confidence score: 5/5

  • This PR has low risk with no critical or high-priority issues identified
  • Score reflects clean code review with only minor suggestions or no issues found
  • Code quality checks passed - safe to proceed with merge

6 files reviewed, 0 comments


Tip: @paragon-run <instructions> to chat with our agent or push fixes!

Dashboard

@deep1401 deep1401 merged commit 52ea1de into main Mar 12, 2026
7 checks passed
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.

2 participants