fix importAttributes is not avaliable in old node versions

[yesmeck]

fix #225

Tested with VSCode 1.91.1 (Node.js 20.9.0) and VSCode 1.94.0 (Node.js 20.16.0).

[apollo-cla]

@yesmeck: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/

[phryneas]

I'm merging in main, as I've added a CI run for 1.90.0 there.

[phryneas]

Generally, could you please add some comment to document that once we hit a minimum of 1.92, most of these workarounds can be removed again, and reference the issue you started?
So we don't confuse future maintainers :)

[phryneas]

Okay, once the tests come back happy all that's left is a changeset - could you please run npx changeset and add a changeset for a patch-level change? :)

[phryneas]

Huh, that test runs fine for me locally. I'll investigate.

[yesmeck]

It seems we can't set contents on importAssertions. It's so weird that it was working for me.

[phryneas]

This is just super weird :(
Any more ideas?
Putting contents into as too? 😢

[yesmeck]

Can you rerun CI? It works for me locally now. We need to set both ⁠format and ⁠contents on the assert object, even though we can’t access the format from ⁠importAssertions, and the order of ⁠format and ⁠contents as keys also matters.

[phryneas]

And we're green.
Thank you very much for your hard work with this!

[phryneas]

Released as /~https://github.com/apollographql/vscode-graphql/releases/tag/v2.3.5