feat: added support for variables and operationName for templates

[peternic]

Added support for variables and operation name to the templates graphql. This will make it possible to use codegen to generate and compiletime check queries.

So need to be able to pass in the variables and the operationName through the sdk.

[codecov]

Codecov Report

Merging #591 (08a64d2) into master (00c73e4) will decrease coverage by 21.28%.
The diff coverage is 87.90%.

@@             Coverage Diff             @@
##           master     #591       +/-   ##
===========================================
- Coverage   98.69%   77.41%   -21.29%     
===========================================
  Files          19       89       +70     
  Lines         764     3090     +2326     
  Branches       51      394      +343     
===========================================
+ Hits          754     2392     +1638     
- Misses         10      697      +687     
- Partials        0        1        +1     

Impacted Files	Coverage Δ
...able/src/api/serviceAccounts/serviceAccountsApi.ts	100.00% <ø> (ø)
...table/src/api/timeSeries/syntheticTimeSeriesApi.ts	100.00% <ø> (ø)
...ackages/stable/src/api/timeSeries/timeSeriesApi.ts	100.00% <ø> (ø)
packages/stable/src/cogniteClient.ts	100.00% <ø> (ø)
packages/stable/src/retryValidator.ts	100.00% <ø> (ø)
packages/stable/src/types.ts	100.00% <ø> (ø)
packages/template/src/__tests__/testUtils.ts	100.00% <ø> (ø)
packages/template/src/cogniteClient.ts	100.00% <ø> (ø)
packages/wells/src/__tests__/testUtils.ts	50.00% <ø> (ø)
packages/wells/src/client/api/casingsApi.ts	11.76% <ø> (ø)
... and 183 more

[stignite]

🚀

[peternic]

After approval, is it correct that I shall merge this as blow to get it released?

feat(templates): added support for variables and operationName for templates [release]

BREAKING CHANGE: The templates runQuery function in beta has changed signature, now accepts an object with query, variables and operationName

[jarlah]

You have degraded test coverage or test coverage is wrong. I don't see any specifically wrong with PR itself ;) Ive seen feat!(module): message floating about but I'm not sure what the ! means yet.

[peternic]

You have degraded test coverage or test coverage is wrong. I don't see any specifically wrong with PR itself ;) Ive seen feat!(module): message floating about but I'm not sure what the ! means yet.

Yes, seems like it is checking against a very old commit for coverage. Not sure how to fix, any ideas?

The ! Is for breaking changes I think...

[polomani]

@peternic BREAKING CHANGE: should work according to https://github.com/conventional-changelog/conventional-changelog/tree/master/packages/conventional-changelog-angular#angular-convention
You could also try to run lerna locally and see if it bumps the number.

We had issues with major releases previously, please reach out if you face it :)

PS. ignore codecov