Conversation

a-tarasyuk

Fixes #26217

@typescript-bottypescript-bot added the For Backlog BugPRs that fix a backlog buglabel Oct 23, 2020
@a-tarasyuka-tarasyuk marked this pull request as ready for review October 25, 2020 13:31
@sandersnsandersn self-assigned this Nov 18, 2020
@DanielRosenwasserDanielRosenwasser mentioned this pull request Nov 19, 2020
Closed
@DanielRosenwasser

@typescript-bot pack this

@typescript-bot

Heya @DanielRosenwasser, I've started to run the tarball bundle task on this PR at dc448f9. You can monitor the build here.

@typescript-bot

Hey @DanielRosenwasser, I've packed this into an installable tgz. You can install it for testing by referencing it in your package.json like so:

{
    "devDependencies": {
        "typescript": "https://typescript.visualstudio.com/cf7ac146-d525-443c-b23c-0d58337efebc/_apis/build/builds/90205/artifacts?artifactName=tgz&fileId=C92047E7D012B4C56A628CDC8A2082B9C79E19DE6093DAD2FAFB6036C022EA4902&fileName=/typescript-4.2.0-insiders.20201202.tgz"
    }
}

and then running npm install.

There is also a playground for this build.

sandersn
sandersn approved these changes Dec 19, 2020
View reviewed changes

Choose a reason for hiding this comment

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

Just a couple of early comments -- I've looked at the tests, and they look good, but haven't looked at the code.

@OliverJAshOliverJAsh mentioned this pull request Dec 29, 2020
Open
5 tasks
jessetrinity
jessetrinity reviewed Jan 5, 2021
View reviewed changes

Choose a reason for hiding this comment

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

I left a couple of questions but otherwise looks great 👍

@jessetrinityjessetrinity merged commit 0c58ede into microsoft:master Jan 9, 2021
Zzzen pushed a commit to Zzzen/TypeScript that referenced this pull request Jan 16, 2021
@a-tarasyuk @Zzzen
feat(26217): Add missing member fix should work for missing function (m…
dfd7d97
…icrosoft#41215)

* feat(26217): Add missing function declaration QF

* use codeFixAvailable instead of codeFix

* add diagnostics messages 'Method not implemented.' and 'Function not implemented.'

* use codeFixAvailable instead of codeFix

* fix signature types
Sign up for free to join this conversation on . Already have an account? Sign in to comment

@sandersn sandersn sandersn approved these changes

@jessetrinity jessetrinity jessetrinity approved these changes

@DanielRosenwasser DanielRosenwasser Awaiting requested review from DanielRosenwasser

@sandersn sandersn

For Backlog BugPRs that fix a backlog bug
PR Backlog
Archived in project
No milestone

Successfully merging this pull request may close these issues.

Add missing member fix should work for missing function