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

Bumping flytestdlib #597

Merged
merged 6 commits into from
Aug 6, 2023
Merged

Bumping flytestdlib #597

merged 6 commits into from
Aug 6, 2023

Conversation

gvashishtha
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

Adds support for Azure sovereign clouds

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

How did you fix the bug, make the feature etc. Link to any design docs etc

Tracking Issue

Remove the 'fixes' keyword if there will be multiple PRs to fix the linked issue

fixes https://github.com/flyteorg/flyte/issues/

Follow-up issue

NA
OR
https://github.com/flyteorg/flyte/issues/

gvashishtha and others added 6 commits August 4, 2023 18:17
Signed-off-by: Gopal K. Vashishtha <gvashishtha@anduril.com>
Signed-off-by: Gopal K. Vashishtha <gvashishtha@anduril.com>
Signed-off-by: Gopal K. Vashishtha <gvashishtha@anduril.com>
Signed-off-by: Gopal K. Vashishtha <gvashishtha@anduril.com>
Signed-off-by: Flyte-Bot <admin@flyte.org>
Co-authored-by: flyte-bot <flyte-bot@users.noreply.github.com>
Signed-off-by: Gopal K. Vashishtha <gvashishtha@anduril.com>
Signed-off-by: Gopal K. Vashishtha <gvashishtha@anduril.com>
@codecov
Copy link

codecov bot commented Aug 6, 2023

Codecov Report

Merging #597 (7bf5da8) into master (4713861) will increase coverage by 1.53%.
Report is 1 commits behind head on master.
The diff coverage is n/a.

❗ Current head 7bf5da8 differs from pull request most recent head ae2a033. Consider uploading reports for the commit ae2a033 to get more accurate results

@@            Coverage Diff             @@
##           master     #597      +/-   ##
==========================================
+ Coverage   58.70%   60.24%   +1.53%     
==========================================
  Files         168      168              
  Lines       16307    13326    -2981     
==========================================
- Hits         9573     8028    -1545     
+ Misses       5884     4448    -1436     
  Partials      850      850              
Flag Coverage Δ
unittests ?

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

see 155 files with indirect coverage changes

@eapolinario eapolinario merged commit d34a940 into flyteorg:master Aug 6, 2023
12 checks passed
eapolinario pushed a commit that referenced this pull request Sep 6, 2023
* using other stow

Signed-off-by: Gopal K. Vashishtha <gvashishtha@anduril.com>

* initial

Signed-off-by: Gopal K. Vashishtha <gvashishtha@anduril.com>

* replacing stdlib

Signed-off-by: Gopal K. Vashishtha <gvashishtha@anduril.com>

* updating both stow and stdlib

Signed-off-by: Gopal K. Vashishtha <gvashishtha@anduril.com>

* Update boilerplate version (#596)

Signed-off-by: Flyte-Bot <admin@flyte.org>
Co-authored-by: flyte-bot <flyte-bot@users.noreply.github.com>
Signed-off-by: Gopal K. Vashishtha <gvashishtha@anduril.com>

* bumping flytestdlib

Signed-off-by: Gopal K. Vashishtha <gvashishtha@anduril.com>

---------

Signed-off-by: Gopal K. Vashishtha <gvashishtha@anduril.com>
Signed-off-by: Flyte-Bot <admin@flyte.org>
Co-authored-by: Flyte Bot <admin@flyte.org>
Co-authored-by: flyte-bot <flyte-bot@users.noreply.github.com>
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