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

Added Example code for new added page of Flyte Type System in examples/basics #1761

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

10sharmashivam
Copy link
Contributor

Tracking issue

Reference Issue
Reference PR

Why are the changes needed?

I added new code example file in example/basics for new page add about Flyte Type System and Data passing. Once this PR is approved then we can use this files raw github user content link and use that to add code example snippets in newly added page for Flyte Type system (about which can be referenced with this PR )

What changes were proposed in this pull request?

Just added new code example file in examples/basics

Check all the applicable boxes

  • I updated the documentation accordingly.
  • All new and existing tests passed.
  • All commits are signed-off.

Related PRs

PR

Docs link

https://flyte--5867.org.readthedocs.build/en/5867/user_guide/basics/flyte_type_system.html

Signed-off-by: 10sharmashivam <10sharmashivam@gmail.com>
@10sharmashivam
Copy link
Contributor Author

Hi @davidmirror-ops, this is the PR about coding examples, which can be used (when approved, and merged) as raw GitHub user content link, serving as code example snippets in the newly added Flyte type system documentation.

Reference PR

Thank you!

@Future-Outlier
Copy link
Member

Hi @davidmirror-ops, this is the PR about coding examples, which can be used (when approved, and merged) as raw GitHub user content link, serving as code example snippets in the newly added Flyte type system documentation.

Reference PR

Thank you!

Hi, will this be merged?
or this will be a blog post.

cc @davidmirror-ops @10sharmashivam

@davidmirror-ops
Copy link
Contributor

@Future-Outlier a blog post. Once it is posted and referenced from the docs, the plan is to close this issue

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