dagu icon indicating copy to clipboard operation
dagu copied to clipboard

[ISSUE #759] Support dynamic git clone for sub-workflows

Open liooooo29 opened this issue 7 months ago • 4 comments

Add a new executor called "git checkout executor" that can checkout a specified branch, commit, or tag from a repository.

  • test repo image image image
  • test checkout branch image image clone result image
  • test checkout tag image image image image
  • test commit image image image

liooooo29 avatar May 16 '25 07:05 liooooo29

Hi @halalala222, thank you for your excellent work on the Git integration. It’s an impressive feature! Just one request, could you please update the PR to target the next branch instead of main? We're currently iterating on next, and merging from main tends to cause some conflicts. Sorry for the trouble, and thanks again!

yottahmd avatar May 16 '25 09:05 yottahmd

ok !I change later

liooooo29 avatar May 16 '25 15:05 liooooo29

Oh sorry it was a mistake. Let's fix the conflict and merge! :)

yottahmd avatar May 30 '25 14:05 yottahmd

Codecov Report

Attention: Patch coverage is 62.89308% with 118 lines in your changes missing coverage. Please review.

Project coverage is 55.69%. Comparing base (b4961b0) to head (2f2c25d).

Files with missing lines Patch % Lines
internal/digraph/executor/checkout.go 62.89% 94 Missing and 24 partials :warning:
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             next     #941      +/-   ##
==========================================
+ Coverage   55.48%   55.69%   +0.20%     
==========================================
  Files          75       76       +1     
  Lines        8240     8626     +386     
==========================================
+ Hits         4572     4804     +232     
- Misses       3281     3404     +123     
- Partials      387      418      +31     
Files with missing lines Coverage Δ
internal/digraph/executor/checkout.go 62.89% <62.89%> (ø)

... and 5 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update b4961b0...2f2c25d. Read the comment docs.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar Jul 12 '25 05:07 codecov[bot]

@halalala222 Sorry, I had to delete the next branch to avoid confusion. Would you mind opening the PR against the main branch when you get a chance? I’d really love to see this excellent feature merged. Thank you!

yottahmd avatar Jul 12 '25 05:07 yottahmd