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

Add return type for _task() #1484

Merged
merged 1 commit into from
Jun 25, 2024
Merged

Add return type for _task() #1484

merged 1 commit into from
Jun 25, 2024

Conversation

wch
Copy link
Collaborator

@wch wch commented Jun 25, 2024

This fixes an issue that is flagged by pyright 1.1.369.

@wch wch enabled auto-merge June 25, 2024 21:29
@wch wch added this pull request to the merge queue Jun 25, 2024
Merged via the queue into main with commit 299f821 Jun 25, 2024
31 checks passed
@wch wch deleted the type-fix branch June 25, 2024 21:42
@wch
Copy link
Collaborator Author

wch commented Jun 25, 2024

I think the error we were seeing with pyright 1.1.369 was due to a bug: microsoft/pyright#8229

schloerke added a commit to machow/py-shiny that referenced this pull request Jul 2, 2024
* main:
  fix(tests): dynamically determine the path to the shiny app (posit-dev#1485)
  tests(deploys): use a stable version of html tools instead of main branch (posit-dev#1483)
  feat(data frame): Support basic cell styling (posit-dev#1475)
  fix: support static files on pyodide / py.cafe under a prefix (posit-dev#1486)
  feat: Dynamic theming (posit-dev#1358)
  Add return type for `_task()` (posit-dev#1484)
  tests(controls): Change API from controls to controller (posit-dev#1481)
  fix(docs): Update path to reflect correct one (posit-dev#1478)
  docs(testing): Add quarto page for testing (posit-dev#1461)
  fix(test): Remove unused testrail reporting from nightly builds (posit-dev#1476)
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.

None yet

1 participant