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

Bumping flytestdlib version #112

Merged
merged 3 commits into from
Aug 6, 2023
Merged

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/

@welcome
Copy link

welcome bot commented Aug 5, 2023

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: Gopal K. Vashishtha <[email protected]>
Signed-off-by: Gopal K. Vashishtha <[email protected]>
Signed-off-by: Gopal K. Vashishtha <[email protected]>
@codecov
Copy link

codecov bot commented Aug 6, 2023

Codecov Report

Merging #112 (6792d98) into master (d5ba84b) will increase coverage by 1.16%.
The diff coverage is n/a.

❗ Current head 6792d98 differs from pull request most recent head a301336. Consider uploading reports for the commit a301336 to get more accurate results

@@            Coverage Diff             @@
##           master     #112      +/-   ##
==========================================
+ Coverage   68.50%   69.67%   +1.16%     
==========================================
  Files          31       31              
  Lines        1667     1385     -282     
==========================================
- Hits         1142      965     -177     
+ Misses        447      342     -105     
  Partials       78       78              
Flag Coverage Δ
unittests ?

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

see 28 files with indirect coverage changes

@eapolinario eapolinario merged commit 74a809f into flyteorg:master Aug 6, 2023
11 checks passed
@welcome
Copy link

welcome bot commented Aug 6, 2023

Congrats on merging your first pull request! 🎉

eapolinario pushed a commit that referenced this pull request Sep 6, 2023
* updating dependencies

Signed-off-by: Gopal K. Vashishtha <[email protected]>

* commit

Signed-off-by: Gopal K. Vashishtha <[email protected]>

* bumping flytestdlib

Signed-off-by: Gopal K. Vashishtha <[email protected]>

---------

Signed-off-by: Gopal K. Vashishtha <[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.

2 participants