Skip to content

Conversation

gezgblack
Copy link
Contributor

@gezgblack gezgblack commented Sep 15, 2020

Proposed changes

This is the destination doc for the integration Insider. Once approved I will add the result of make catalog and I will merge. Let me know if there's any improvement you see @sanscontext, thank you.

Copy link
Contributor

@sanscontext sanscontext left a comment

Choose a reason for hiding this comment

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

Some edits, and a question or two. Overall, looks good, please tell the Insider team I said so. :)

sanscontext and others added 8 commits September 15, 2020 13:29
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
@gezgblack
Copy link
Contributor Author

Thank you for the review @sanscontext. Let me know if there are any other changes or if you're happy enough with that and I add the yaml and merge.

@sanscontext
Copy link
Contributor

One more item (thanks for checking on that!) and then we're good!

Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
@gezgblack
Copy link
Contributor Author

One more item (thanks for checking on that!) and then we're good!

Resolved now :) will be waiting for approval.

@gezgblack gezgblack merged commit 7aed962 into master Sep 18, 2020
@gezgblack gezgblack deleted the insider/docs branch September 18, 2020 09:59
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