Skip to content

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

Merged
leoluz merged 2 commits intoargoproj:masterfrom
crenshaw-dev:fix-appset-docs
Aug 9, 2022
Merged

docs: revert appset path.path and path.segments docs#10254
leoluz merged 2 commits intoargoproj:masterfrom
crenshaw-dev:fix-appset-docs

Conversation

@crenshaw-dev
Copy link
Member

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

All modified and coverable lines are covered by tests ✅

Project coverage is 46.18%. Comparing base (915bd81) to head (bd5af55).
Report is 2651 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #10254   +/-   ##
=======================================
  Coverage   46.18%   46.18%           
=======================================
  Files         226      226           
  Lines       27581    27581           
=======================================
  Hits        12737    12737           
  Misses      13124    13124           
  Partials     1720     1720           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? 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
Member 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
Member 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.

3 participants