Skip to content

Revert "Validate Ksonnet apps through component dir presence (#708)"#730

Merged
alexmt merged 1 commit intoargoproj:masterfrom
merenbach:revert-594-better-ksonnet-detection
Oct 29, 2018
Merged

Revert "Validate Ksonnet apps through component dir presence (#708)"#730
alexmt merged 1 commit intoargoproj:masterfrom
merenbach:revert-594-better-ksonnet-detection

Conversation

@merenbach
Copy link
Contributor

This reverts commit 1844be6 from #708, which has demonstrated output inconsistent with the testing performed.

@codecov-io
Copy link

codecov-io commented Oct 27, 2018

Codecov Report

Merging #730 into master will decrease coverage by 0.01%.
The diff coverage is 37.5%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #730      +/-   ##
==========================================
- Coverage   22.05%   22.04%   -0.02%     
==========================================
  Files          40       40              
  Lines        6288     6287       -1     
==========================================
- Hits         1387     1386       -1     
  Misses       4714     4714              
  Partials      187      187
Impacted Files Coverage Δ
reposerver/repository/repository.go 20.37% <37.5%> (-0.3%) ⬇️

Continue to review full report at Codecov.

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

@merenbach merenbach requested review from alexmt and jessesuen October 29, 2018 06:15
@alexmt alexmt merged commit a0b5af0 into argoproj:master Oct 29, 2018
ppapapetrou76 pushed a commit to ppapapetrou76/argo-cd that referenced this pull request Aug 19, 2025
…#708)" (argoproj#730)

* Revert "Add option to skip the dryrun from the sync context (argoproj#708)"

This reverts commit 717b8bf.

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>

* format

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>

---------

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Co-authored-by: Alexandre Gaudreault <alexandre_gaudreault@intuit.com>
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