-
Notifications
You must be signed in to change notification settings - Fork 28
Type cleanup #567
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
Type cleanup #567
Changes from all commits
Commits
Show all changes
77 commits
Select commit
Hold shift + click to select a range
2bef103
Runtime code fixes
wch 5c60398
Remove six dependency
wch e08f9c1
Fixes for http_support
wch 8dc65a4
Rename TaskStatus to TaskStatusV0
wch 1c4a828
Fixes for models.py
wch 6e564b0
Fixes for json_web_token
wch 833ca0c
Fixes for log
wch c3238d0
Remove click.get_current_context() check
wch 58be270
Fixes in actions_content
wch 210cd2b
Fixes in api
wch 45cfe00
Consolidate ApiAppRecord and ContentItem
wch d3914e6
Fixes for api
wch 1b2e2f4
Update _handle_bad_response to return input and do type narrowing
wch e83b8fa
Replace ContentItem with ContentItemV0 and ContentItemV1
wch 66e9515
Add check for None
wch de79b59
Add TaskStatusV1
wch 10ba57d
Fixes in actions_content
wch d970e57
Make Environment fields non-Optional
wch 2f1d758
Add typing for app_upload
wch a5cddd9
Add pyright ignore comments
wch 7fc2f34
Add accessor function for content_build_store
wch 319a053
Convert Environment to dataclass
wch 28ca0aa
Add type for ServerSettings
wch 29a8b32
Add PyInfo type
wch a7ecbc3
Add types for cache API
wch e9c4057
Add type for app_config
wch 09f814b
Add type for bootstrap
wch e068cb8
Small type fixes
wch 0f12e1f
Update ContentItemV0 and ContentItemV1
wch edb125e
Fix bootstrapdto formatting
wch a3ae499
Small typing fix
wch 7161e3a
Use Mapping type
wch a2a9f85
Remove checks for string encode method
wch 1cb23c5
Fixes for TaskStatusV0 and BuildOutputDTO
wch fdba481
Switch from mypy to pyright
wch c82d4a6
Small type fixes
wch 8050031
Enable CI for pull request
wch 4ec3639
Remove unused import
wch 483076b
Add types for PositClient API
wch 407e6db
More API typing
wch dfc5ae2
Change init_content_build_store to ensure_content_build_store
wch 11f7fab
Remove reset() and drop_context() methods
wch d74e8c6
Check class of client instead of using cast()
wch 2fea71e
Remove unused import
wch 9f29ef5
Remove unused deprecated functions
wch 2b42944
Remove unused functions from actions.py
wch c318698
Refactor RSConnectExecutor and make all args explicit
wch 2d9424b
Switch type from BinaryIO to IO[bytes]
wch 2925857
Small cleanups
wch 71d5f75
Remove unused functions
wch f81f3ab
In handle_bad_response(), add is_httpresponse param
wch dce878f
Use explicit args instead of locals()
wch d003107
Small cleanups
wch d67dada
Remove unnecessary @property
wch 3b395e0
Rename type
wch 59b63a0
Move contents of state dictionary into RSConnectExecutor class
wch a70b4ab
Use Paramspec for make_bundle
wch efe8fa0
Remove unused kwargs
wch cf1f2fd
Cleanup for validate_server()
wch 94ee0ab
Remove unused params for validate_server()
wch 8fd6cc3
Fixes for actions_content.py
wch da40b48
Fixes for bundly.py
wch a19bce8
Cleanups in api.py
wch 497e250
Add comment
wch 26e0353
Add leading - to make command
wch a0f1ed2
Typing fixes
wch f5ca6e5
Fix for LoggerAdapter not being generic in Python<=3.11
wch 76a871a
Fixes for Python 3.8
wch b4ebd2a
Fixes for unit tests
wch 2ed8b70
Remove unused import
wch da5c68d
Use Union instead of | for Python 3.8
wch 8e350e0
More uses of Union
wch be37379
Use List instead of list for Python 3.8
wch 7bcc14b
Don't handle bad responses in RSConnectClient.bootstrap()
wch d1df87c
Add VS Code settings file
wch 89f855d
Update pyproject.toml
wch 667795f
Revert change to GHA workflow
wch File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| { | ||
| "recommendations": [ | ||
| "ms-python.vscode-pylance", | ||
| "ms-python.black-formatter" | ||
| ] | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,27 @@ | ||
| { | ||
| "files.trimTrailingWhitespace": true, | ||
| "files.insertFinalNewline": true, | ||
| "editor.tabSize": 4, | ||
| "files.encoding": "utf8", | ||
| "files.eol": "\n", | ||
| "[python]": { | ||
| "editor.defaultFormatter": "ms-python.black-formatter", | ||
| "editor.formatOnSave": true, | ||
| "editor.tabSize": 4, | ||
| "editor.codeActionsOnSave": { | ||
| "source.organizeImports": "explicit" | ||
| }, | ||
| "editor.rulers": [120], | ||
| }, | ||
| "isort.args": ["--profile", "black"], | ||
| "files.exclude": { | ||
| "**/__pycache__": true, | ||
| "build/**": true, | ||
| "venv/**": true, | ||
| }, | ||
| "search.exclude": { | ||
| "**/__pycache__": true, | ||
| "build/**": true, | ||
| "venv/**": true, | ||
| }, | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.