Skip to content
This repository has been archived by the owner on Oct 23, 2023. It is now read-only.

Introduce TypeAnnotation#minor #232

Merged
merged 5 commits into from
Feb 8, 2022
Merged

Conversation

kennyworkman
Copy link
Contributor

Read then delete this section

- Make sure to use a concise title for the pull-request.

- Use #patch, #minor or #major in the pull-request title to bump the corresponding version. Otherwise, the patch version
will be bumped. More details

TL;DR

Introduce new message to idl for typing annotations re: flyteorg/flyte#1855

Type

  • Bug Fix
  • Feature
  • Plugin

Are all requirements met?

  • Code completed
  • Smoke tested
  • Unit tests added
  • Code documentation added
  • Any pending items have an associated Issue

Complete description

flyteorg/flyte#1855

@welcome
Copy link

welcome bot commented Nov 30, 2021

Thank you for opening this pull request! 🙌

These tips will help get your PR across the finish line:

  • Most of the repos have a PR template; if not, fill it out to the best of your knowledge.
  • Sign off your commits (Reference: DCO Guide).

Signed-off-by: Kenny Workman <[email protected]>
EngHabu
EngHabu previously approved these changes Nov 30, 2021
Copy link
Contributor

@EngHabu EngHabu left a comment

Choose a reason for hiding this comment

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

Looks great! In terms of implementation, let's get all PRs ready before we start merging. This enables you to make small modifications as we learn more while implementing the feature and avoids destabilizing the main branch until the feature is ready...

protos/flyteidl/core/types.proto Show resolved Hide resolved
@kennyworkman
Copy link
Contributor Author

sounds good

Co-authored-by: Haytham Abuelfutuh <[email protected]>
Signed-off-by: Kenny Workman <[email protected]>
@eapolinario eapolinario mentioned this pull request Dec 3, 2021
7 tasks
Signed-off-by: Kenny Workman <[email protected]>
@kennyworkman kennyworkman force-pushed the master branch 2 times, most recently from 8c4bb8c to 17fc33f Compare December 10, 2021 04:03
Copy link
Contributor

@eapolinario eapolinario left a comment

Choose a reason for hiding this comment

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

@EngHabu , can you also take another look?

@kennyworkman kennyworkman merged commit ea09b27 into flyteorg:master Feb 8, 2022
@welcome
Copy link

welcome bot commented Feb 8, 2022

Congrats on merging your first pull request! 🎉

eapolinario pushed a commit that referenced this pull request Sep 8, 2023
* feat: introduce TypeAnnotation

Signed-off-by: Kenny Workman <[email protected]>

* fix: message description

Co-authored-by: Haytham Abuelfutuh <[email protected]>
Signed-off-by: Kenny Workman <[email protected]>

* fix: regen proto after merge

Signed-off-by: Kenny Workman <[email protected]>

Co-authored-by: Haytham Abuelfutuh <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants