Skip to content

Patch to support multi-source apps.#100

Merged
ivanklee86 merged 1 commit intomainfrom
patch_multi-source_apps
Feb 11, 2025
Merged

Patch to support multi-source apps.#100
ivanklee86 merged 1 commit intomainfrom
patch_multi-source_apps

Conversation

@ivanklee86
Copy link
Copy Markdown
Owner

Patch live revision field for multi-source apps (since manifests doesn't seem to support it).

@github-actions
Copy link
Copy Markdown
Contributor

Test Results

32 tests  ±0   32 ✅ ±0   3s ⏱️ ±0s
 3 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit bf8c544. ± Comparison against base commit e1563c9.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Feb 11, 2025

Codecov Report

❌ Patch coverage is 71.42857% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 62.50%. Comparing base (e1563c9) to head (bf8c544).
⚠️ Report is 92 commits behind head on main.

Files with missing lines Patch % Lines
internal/argocd/wrapper.go 71.42% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #100      +/-   ##
==========================================
+ Coverage   62.45%   62.50%   +0.04%     
==========================================
  Files           8        8              
  Lines         562      568       +6     
==========================================
+ Hits          351      355       +4     
- Misses        211      213       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

@ivanklee86 ivanklee86 merged commit ed84a41 into main Feb 11, 2025
@ivanklee86 ivanklee86 deleted the patch_multi-source_apps branch February 11, 2025 11:53
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