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

chore: typing decorators #1447

Merged
merged 6 commits into from
Sep 2, 2024

Conversation

jorwoods
Copy link
Contributor

Decorators like api, parameter_added_in, and _tsrequest_wrapped altered the typing signatures and obfuscated mypy from inspecting any of the functions wrapped by these decorators. This PR aims to type these decorators, and fix any underlying typing issues revealed by fixing this.

@jorwoods jorwoods marked this pull request as ready for review August 24, 2024 17:27
@jorwoods jorwoods mentioned this pull request Aug 29, 2024
Copy link
Contributor

@jacalata jacalata left a comment

Choose a reason for hiding this comment

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

looks great

@jacalata jacalata merged commit 678d46a into tableau:development Sep 2, 2024
22 checks passed
@jorwoods jorwoods deleted the jorwoods/typing_decorators branch September 2, 2024 19:30
jacalata added a commit that referenced this pull request Sep 17, 2024
v0.33

Features:
- add name, datasource-name to Job item
- enable bulk add and remove users
- Linked Tasks: get, get by ID, run Now
- implement Tags: create new, add/delete for workbooks, flows, datasources
- get page and chunk size from env vars
- add some repr implementations
- implement virtual connections

Bugfix:
- #1447
- #1449
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants