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

TypeScript more type definitions captures #2916

Merged
merged 5 commits into from
Nov 14, 2024
Merged

Conversation

tomasz-stefaniak
Copy link
Collaborator

Description

Captures nested return types in methods and functions, captures generic return types and arguments in methods.

Checklist

  • [] The relevant docs, if any, have been updated or created

Screenshots

[ For visual changes, include screenshots. ]

Testing

[ For new or modified features, provide step-by-step testing instructions to validate the intended behavior of the change. ]
[ For new UI features, ensure that the changes look good across viewport widths, light/dark theme, etc ]

Copy link

netlify bot commented Nov 14, 2024

Deploy Preview for continuedev failed. Why did it fail? →

Name Link
🔨 Latest commit 5de52c8
🔍 Latest deploy log https://app.netlify.com/sites/continuedev/deploys/673646e753b3af00088448fd

sestinj
sestinj previously approved these changes Nov 14, 2024
@sestinj
Copy link
Contributor

sestinj commented Nov 14, 2024

@tomasz-stefaniak I just realized that one of the tests is failing. I don't actually know for certain whether it was this PR that caused it. And tempting as it is, I want to try and build the habit that we never merge anything if there is a test failing. So can just wait until tomorrow morning to merge and make the new pre-release

@sestinj sestinj merged commit 2f089eb into main Nov 14, 2024
2 of 6 checks passed
@sestinj sestinj deleted the tomasz/more-ts-context branch November 14, 2024 18:59
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.

2 participants