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

Test pr 799 #801

Merged
merged 8 commits into from
May 11, 2023
Merged

Test pr 799 #801

merged 8 commits into from
May 11, 2023

Conversation

sfc-gh-aalam
Copy link
Contributor

@sfc-gh-aalam sfc-gh-aalam commented Apr 21, 2023

Please answer these questions before submitting your pull requests. Thanks!

  1. What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.

    Fixes SNOW-799415: Wrong type definitions (newtype instead of alias) snowflake-connector-python#1525 and SNOW-801097: Pyre type checker returns unexpected type errors with snowpark-python #810

  2. Fill out the following pre-review checklist:

    • I am adding a new automated test(s) to verify correctness of my new code
    • I am adding new logging messages
    • I am adding a new telemetry message
    • I am adding new credentials
    • I am adding a new dependency
  3. Please describe how your code solves the related issue.

    Test PR for user.

@codecov
Copy link

codecov bot commented Apr 21, 2023

Codecov Report

Merging #801 (b58af36) into main (7bfbaa4) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #801   +/-   ##
=======================================
  Coverage   98.09%   98.09%           
=======================================
  Files          49       49           
  Lines        8403     8403           
  Branches     1425     1425           
=======================================
  Hits         8243     8243           
  Misses         92       92           
  Partials       68       68           
Impacted Files Coverage Δ
src/snowflake/snowpark/_internal/type_utils.py 97.90% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@sfc-gh-aalam sfc-gh-aalam requested review from a team, sfc-gh-jdu and sfc-gh-stan and removed request for a team April 22, 2023 01:35
@sfc-gh-aalam sfc-gh-aalam added NO-CHANGELOG-UPDATES This pull request does not need to update CHANGELOG.md and removed NO-CHANGELOG-UPDATES This pull request does not need to update CHANGELOG.md labels Apr 23, 2023
@github-actions
Copy link

github-actions bot commented Apr 25, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@sfc-gh-mkeller
Copy link
Collaborator

@StefanUlbrich Could you please sign our CLA? We'd be happy to merge your PR afterwards!

@StefanUlbrich
Copy link
Contributor

I have read the CLA Document and I hereby sign the CLA

@StefanUlbrich
Copy link
Contributor

@StefanUlbrich Could you please sign our CLA? We'd be happy to merge your PR afterwards!

sorry for the delay. Thanks for reviewing

Copy link
Collaborator

@sfc-gh-mkeller sfc-gh-mkeller left a comment

Choose a reason for hiding this comment

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

🚢 🚢

CHANGELOG.md Outdated Show resolved Hide resolved
remove shoutouts from changelog as it is not a consistent practice in the team
@sfc-gh-mkeller sfc-gh-mkeller self-assigned this May 3, 2023
@sfc-gh-mkeller sfc-gh-mkeller merged commit 183d2a4 into main May 11, 2023
@sfc-gh-mkeller sfc-gh-mkeller deleted the test_pr_799 branch May 11, 2023 00:48
@github-actions github-actions bot locked and limited conversation to collaborators May 11, 2023
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.

SNOW-799415: Wrong type definitions (newtype instead of alias)
3 participants