Skip to content

Conversation

stefmolin
Copy link
Contributor

Description

Added missing py.typed file. Fixes #1106

Checklist

  • Add test cases to all the changes you introduce
  • Run ./scripts/format and ./scripts/test locally to ensure this change passes linter check and test
  • Test the changes on the local machine manually
  • Update the documentation for the changes

Expected behavior

Steps to Test This Pull Request

Additional context

@stefmolin
Copy link
Contributor Author

The checks are failing on untouched files (images in the docs).

@Lee-W
Copy link
Member

Lee-W commented May 20, 2024

The checks are failing on untouched files (images in the docs).

Yep, that will be fixed by #1126

Copy link

codecov bot commented May 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.58%. Comparing base (120d514) to head (a7d4189).
Report is 325 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1127      +/-   ##
==========================================
+ Coverage   97.33%   97.58%   +0.24%     
==========================================
  Files          42       55      +13     
  Lines        2104     2486     +382     
==========================================
+ Hits         2048     2426     +378     
- Misses         56       60       +4     
Flag Coverage Δ
unittests 97.58% <ø> (+0.24%) ⬆️

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

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

Copy link
Member

@Lee-W Lee-W left a comment

Choose a reason for hiding this comment

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

@stefmolin I think this is not really a fix we need to bump a version and include the changelog. Could you please change the type in the commit message to style instead?

@Lee-W Lee-W added pr-status: ready-to-merge almost ready to merge. just keep it for a few days for others to check pr-status: wait-for-modification and removed pr-status: wait-for-review labels May 20, 2024
@Lee-W Lee-W merged commit a9f86be into commitizen-tools:master May 20, 2024
@Lee-W
Copy link
Member

Lee-W commented May 20, 2024

@stefmolin Thank you for your quick update 😄

@stefmolin stefmolin deleted the py.typed branch May 20, 2024 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-status: ready-to-merge almost ready to merge. just keep it for a few days for others to check pr-status: wait-for-modification pr-status: wait-for-review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add py.typed marker
3 participants