-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
feat: Merge applicationset docs and examples #8955
feat: Merge applicationset docs and examples #8955
Conversation
Signed-off-by: rishabh625 <rishabhmishra625@gmail.com>
Signed-off-by: rishabh625 <rishabhmishra625@gmail.com>
Codecov Report
@@ Coverage Diff @@
## master #8955 +/- ##
==========================================
+ Coverage 43.40% 44.92% +1.52%
==========================================
Files 186 212 +26
Lines 23373 25264 +1891
==========================================
+ Hits 10145 11350 +1205
- Misses 11779 12310 +531
- Partials 1449 1604 +155
Continue to review full report at Codecov.
|
@jannfis @alexmt @crenshaw-dev : Please 🙏 review this |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall, looks awesome!
A few top-level things:
- I see a few absolute links that could be changed to relative. I think you can find them all with a search for
https://argo-cd.readthedocs.io
. - I'm getting an error on
make serve-docs-local
regarding thegoogle_analytics
config option.
@@ -0,0 +1,10 @@ | |||
https://1.2.3.4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since we don't run docs linting in this repo, can we just delete this file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is there future plans to do doc linting in this repo?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dmolik we disabled it because it was extremely flaky, and the tool we were using wasn't flexible enough to configure away the flakiness.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks!
Signed-off-by: rishabh625 <rishabhmishra625@gmail.com>
…set to argocd,removed unused file Signed-off-by: rishabh625 <rishabhmishra625@gmail.com>
36d949d
to
dcff73e
Compare
Signed-off-by: rishabh625 <rishabhmishra625@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
Will wait for one more review, since it's a big PR. :-) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM as well! Thanks @rishabh625
* feat: Migrate applicationset docs,examples into argocd Signed-off-by: rishabh625 <rishabhmishra625@gmail.com> * corrected applicationset binary name in manifest present in doc Signed-off-by: rishabh625 <rishabhmishra625@gmail.com> * corrected autogenerated file Signed-off-by: rishabh625 <rishabhmishra625@gmail.com> * Included gettimg started page, added relative lookup from applicationset to argocd,removed unused file Signed-off-by: rishabh625 <rishabhmishra625@gmail.com> * commiting michael suggestion of text Signed-off-by: rishabh625 <rishabhmishra625@gmail.com> Signed-off-by: asingh51 <Ashutosh_Singh@intuit.com>
* feat: Migrate applicationset docs,examples into argocd Signed-off-by: rishabh625 <rishabhmishra625@gmail.com> * corrected applicationset binary name in manifest present in doc Signed-off-by: rishabh625 <rishabhmishra625@gmail.com> * corrected autogenerated file Signed-off-by: rishabh625 <rishabhmishra625@gmail.com> * Included gettimg started page, added relative lookup from applicationset to argocd,removed unused file Signed-off-by: rishabh625 <rishabhmishra625@gmail.com> * commiting michael suggestion of text Signed-off-by: rishabh625 <rishabhmishra625@gmail.com> Signed-off-by: wojtekidd <wojtek.cichon@protonmail.com>
Note on DCO:
If the DCO action in the integration test fails, one or more of your commits are not signed off. Please click on the Details link next to the DCO action for instructions on how to resolve this.
Checklist:
Related #8864
This PR copy's examples and documentation of applicationset into argocd, have changed text of documentation wherever required please review with extra attention as may have mistakes.