Skip to content

patch: Indicate that the package is typed #124

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

Merged
merged 1 commit into from
May 13, 2024
Merged

Conversation

jmgate
Copy link
Collaborator

@jmgate jmgate commented May 9, 2024

Type: Task

Description

According to PEP 561, we need to add this file to static type checkers can infer the types from the package.

Motivation

When working on open-sourcing staged-script, I noticed mypy wasn't pulling in type information from this package.

According to PEP 561, we need to add this file to static type checkers
can infer the types from the package.
@jmgate jmgate added the Type: Task Something we need to do that doesn't fit any other type. label May 9, 2024
Copy link

codecov bot commented May 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.04%. Comparing base (05e5353) to head (bd89c55).

Additional details and impacted files
@@             Coverage Diff             @@
##           master     #124       +/-   ##
===========================================
+ Coverage        0   94.04%   +94.04%     
===========================================
  Files           0        2        +2     
  Lines           0      168      +168     
  Branches        0       43       +43     
===========================================
+ Hits            0      158      +158     
- Misses          0        4        +4     
- Partials        0        6        +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jmgate jmgate self-assigned this May 9, 2024
Copy link
Collaborator

@william76 william76 left a comment

Choose a reason for hiding this comment

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

LGTM

@jmgate jmgate merged commit 98268fc into master May 13, 2024
13 checks passed
@jmgate jmgate deleted the add-py-typed-file branch May 13, 2024 13:46
@jmgate
Copy link
Collaborator Author

jmgate commented May 13, 2024

I can confirm that the 1.0.8 release made mypy happy as far as staged-script is concerned, so adding this one file did the trick.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Task Something we need to do that doesn't fit any other type.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants