Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: revert appset path.path and path.segments docs #10254

Merged
merged 2 commits into from Aug 9, 2022

Conversation

crenshaw-dev
Copy link
Collaborator

I think the find/replace when just slightly too far in #10026.

@speedfl could you take a look and confirm that this reversion is correct?

Signed-off-by: CI <michael@crenshaw.dev>
@codecov
Copy link

codecov bot commented Aug 9, 2022

Codecov Report

Merging #10254 (bd5af55) into master (915bd81) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #10254   +/-   ##
=======================================
  Coverage   46.18%   46.18%           
=======================================
  Files         226      226           
  Lines       27581    27581           
=======================================
  Hits        12737    12737           
  Misses      13124    13124           
  Partials     1720     1720           
Impacted Files Coverage Δ
applicationset/generators/git.go 85.96% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@speedfl
Copy link
Contributor

speedfl commented Aug 9, 2022

Ah yes I remember, I did this change and finally we added a dedicated section for GoTemplate. However I did not rollback changes

Copy link
Contributor

@speedfl speedfl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok for me.

@crenshaw-dev
Copy link
Collaborator Author

Thank you @speedfl!

Copy link
Contributor

@speedfl speedfl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Signed-off-by: CI <michael@crenshaw.dev>
@crenshaw-dev
Copy link
Collaborator Author

Good catch, fixed!

@leoluz leoluz enabled auto-merge (squash) August 9, 2022 20:58
@leoluz leoluz merged commit 3d06d82 into argoproj:master Aug 9, 2022
@crenshaw-dev crenshaw-dev deleted the fix-appset-docs branch August 9, 2022 22:33
ashutosh16 pushed a commit to ashutosh16/argo-cd that referenced this pull request Aug 11, 2022
* docs: revert appset path.path and path.segments docs

Signed-off-by: CI <michael@crenshaw.dev>

* more

Signed-off-by: CI <michael@crenshaw.dev>
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.

None yet

3 participants