Skip to content

fix(typing): Annotate pipe first argument type as GT#757

Merged
rich-iannone merged 4 commits intoposit-dev:mainfrom
FBruzzesi:fix/typing-pipe
Feb 12, 2026
Merged

fix(typing): Annotate pipe first argument type as GT#757
rich-iannone merged 4 commits intoposit-dev:mainfrom
FBruzzesi:fix/typing-pipe

Conversation

@FBruzzesi
Copy link
Contributor

Summary

Very tiny change: func callable in pipe function was missing the first argument annotation, leading to the following:

Return type is unknownPylance[reportUnknownVariableType]

Expected 0 positional arguments

screenshot image

By adding "GT" via Concatenate, the issue is resolved.

Checklist

@codecov
Copy link

codecov bot commented Aug 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.31%. Comparing base (84bd229) to head (33f8c0d).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #757   +/-   ##
=======================================
  Coverage   92.31%   92.31%           
=======================================
  Files          49       49           
  Lines        6000     6000           
=======================================
  Hits         5539     5539           
  Misses        461      461           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@FBruzzesi FBruzzesi changed the title fix(typing): pipe missing first argument type fix(typing): pipe has incorrect type for first argument (GT) Aug 9, 2025
@FBruzzesi FBruzzesi changed the title fix(typing): pipe has incorrect type for first argument (GT) fix(typing): Annotate pipe first argument type as GT Aug 9, 2025
@machow machow self-assigned this Aug 18, 2025
Copy link
Collaborator

@machow machow left a comment

Choose a reason for hiding this comment

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

Sorry for the long wait -- we're working on getting through PRs! This LGTM. @rich-iannone do you want to merge?

@machow machow assigned rich-iannone and unassigned machow Feb 12, 2026
@rich-iannone rich-iannone merged commit ba9bc00 into posit-dev:main Feb 12, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants