Skip to content

Add list type safety, helpers; remove cruft #157

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 4 commits into from
Feb 7, 2022
Merged

Add list type safety, helpers; remove cruft #157

merged 4 commits into from
Feb 7, 2022

Conversation

drew-512
Copy link
Contributor

@drew-512 drew-512 commented Feb 3, 2022

No description provided.

@codecov
Copy link

codecov bot commented Feb 3, 2022

Codecov Report

Merging #157 (2ee3956) into master (3068cb5) will decrease coverage by 0.02%.
The diff coverage is 42.85%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #157      +/-   ##
==========================================
- Coverage   74.55%   74.53%   -0.03%     
==========================================
  Files          64       64              
  Lines       11187    11190       +3     
==========================================
- Hits         8341     8340       -1     
- Misses       2297     2301       +4     
  Partials      549      549              
Impacted Files Coverage Δ
py/traceback.go 22.22% <ø> (-2.11%) ⬇️
py/list.go 62.86% <33.33%> (-1.24%) ⬇️
py/dict.go 77.31% <100.00%> (+0.23%) ⬆️

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 3068cb5...2ee3956. Read the comment docs.

Copy link
Member

@sbinet sbinet left a comment

Choose a reason for hiding this comment

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

LGTM

@sbinet sbinet merged commit 137c43f into go-python:master Feb 7, 2022
@drew-512 drew-512 deleted the add-type-safety-and-essentials branch February 17, 2022 02:03
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants