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

Move deprecated integrations docs to flyte #5283

Merged
merged 2 commits into from
Apr 25, 2024

Conversation

neverett
Copy link
Contributor

@neverett neverett commented Apr 24, 2024

What changes were proposed in this pull request?

Moves deprecated integrations docs to flyte from flytesnacks and uses rli (remoteliteralinclude) directive to include code from code files in flytesnacks repo.

Check all the applicable boxes

  • All new and existing tests passed.
  • All commits are signed-off.

Related PRs

Merge this PR first, then merge flyteorg/flytesnacks#1655

Docs link

https://flyte--5283.org.readthedocs.build/en/5283/deprecated_integrations/index.html

Signed-off-by: nikki everett <nikki@union.ai>
@neverett neverett requested a review from ppiegaze as a code owner April 24, 2024 18:15
@neverett neverett self-assigned this Apr 24, 2024
Signed-off-by: nikki everett <nikki@union.ai>
Copy link
Member

@ppiegaze ppiegaze left a comment

Choose a reason for hiding this comment

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

LGTM

@neverett neverett merged commit b79cd51 into master Apr 25, 2024
46 checks passed
@neverett neverett deleted the nikki/docs/deprecated-integrations branch April 25, 2024 14:31
austin362667 pushed a commit to austin362667/flyte that referenced this pull request May 7, 2024
* move deprecated integrations docs to flyte

Signed-off-by: nikki everett <nikki@union.ai>

* add redirect, exclude deprecated_integrations file

Signed-off-by: nikki everett <nikki@union.ai>

---------

Signed-off-by: nikki everett <nikki@union.ai>
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

2 participants