TypeScript icon indicating copy to clipboard operation
TypeScript copied to clipboard

Add string literal completions for `package.json` `imports` field

Open Andarist opened this issue 1 year ago • 7 comments

closes https://github.com/microsoft/TypeScript/issues/52460 closes https://github.com/microsoft/TypeScript/issues/57680 closes https://github.com/microsoft/TypeScript/issues/57777

Currently, this only has tests based on https://github.com/microsoft/TypeScript/pull/55015 but I still have to add more

Andarist avatar Mar 11 '24 10:03 Andarist

@andrewbranch I opened this as a draft because there are some minor cleanups to be done here. I'd appreciate an early review here though - in case there is something fundamentally wrong with this.

Andarist avatar Mar 11 '24 10:03 Andarist

@Andarist did you still want to pursue this? We're looking to get this done in the TS 5.7 timeframe.

DanielRosenwasser avatar Aug 23 '24 00:08 DanielRosenwasser

@DanielRosenwasser yes, it would be great if @andrewbranch could give this a quick look to check if im not doing anything overly wrong so I dont spend too much time on cleaning up things that will turn out to be wrong in the end ;p

Andarist avatar Aug 23 '24 05:08 Andarist

@andrewbranch thanks for the review! I’ll try to clean this up asap

Andarist avatar Sep 25 '24 05:09 Andarist

The TypeScript team hasn't accepted the linked issue #52460. If you can get it accepted, this PR will have a better chance of being reviewed.

typescript-bot avatar Oct 10 '24 09:10 typescript-bot

I have been busy the last 2 weeks, I've done some progress on this in the past days but I hope to get everything done till the end of this week 🤞

Andarist avatar Oct 10 '24 09:10 Andarist

@andrewbranch it's ready for re-review :)

I could also use a build of this to test it more easily in a real project, and not only in the test harness ;p cc @jakebailey

Andarist avatar Oct 17 '24 10:10 Andarist

@typescript-bot pack this

jakebailey avatar Oct 31 '24 19:10 jakebailey

Starting jobs; this comment will be updated as builds start and complete.

Command Status Results
pack this ✅ Started ✅ Results

typescript-bot avatar Oct 31 '24 19:10 typescript-bot

Hey @jakebailey, 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/164040/artifacts?artifactName=tgz&fileId=7D1CD9129128DE0FC3646A76F014BB023412AEA343119330949BF42DF191661C02&fileName=/typescript-5.7.0-insiders.20241031.tgz"
    }
}

and then running npm install.


There is also a playground for this build and an npm module you can use via "typescript": "npm:@typescript-deploys/[email protected]".;

typescript-bot avatar Oct 31 '24 19:10 typescript-bot