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

Pickup compiler change to trim type metadata fields #430

Merged
merged 1 commit into from
May 24, 2022
Merged

Conversation

EngHabu
Copy link
Contributor

@EngHabu EngHabu commented May 24, 2022

Signed-off-by: Haytham Abuelfutuh [email protected]

TL;DR

Pickup flytepropeller's compiler changes to strip type's metadata fields

Type

  • Bug Fix
  • Feature
  • Plugin

Tracking Issue

fixes flyteorg/flyte#2516

@EngHabu EngHabu marked this pull request as ready for review May 24, 2022 03:22
@codecov
Copy link

codecov bot commented May 24, 2022

Codecov Report

Merging #430 (779c67f) into master (b1ab36c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #430   +/-   ##
=======================================
  Coverage   61.33%   61.33%           
=======================================
  Files         156      156           
  Lines       11095    11095           
=======================================
  Hits         6805     6805           
  Misses       3586     3586           
  Partials      704      704           
Flag Coverage Δ
unittests 60.25% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b1ab36c...779c67f. Read the comment docs.

@EngHabu EngHabu enabled auto-merge (squash) May 24, 2022 04:25
@EngHabu EngHabu merged commit b0ccd98 into master May 24, 2022
@EngHabu EngHabu deleted the prop-compiler branch May 24, 2022 05:18
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.

[Core feature] Server-side compiler should strip Type Metadata
2 participants