-
Notifications
You must be signed in to change notification settings - Fork 25
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
feat: deprecating the dynamic api
entrypoint
#723
Conversation
@@ -137,72 +128,6 @@ describe('httpsnippet-client-api', () => { | |||
|
|||
expect(`${code}\n`).toStrictEqual(expected); | |||
}); | |||
|
|||
it('should generate a functional snippet', async () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These tests used a dist of the dynamic api
entrypoint. I don't really have any interest in trying to get this extremely gross (my fault) vm
sandboxing work working on that so I'm deleting all this. In the future it would probably be worth fleshing out a full integration suite for all these test cases but that's all really pertinent to api
and not code snippet generation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would be cool to have some sort of mocking framework/library for easily testing these plugins
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah that'd be nice
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you'll be missed, proxy variant 🫡
"main": "./dist/index.js", | ||
"types": "./dist/index.d.ts", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:rip-cowboy:
@@ -137,72 +128,6 @@ describe('httpsnippet-client-api', () => { | |||
|
|||
expect(`${code}\n`).toStrictEqual(expected); | |||
}); | |||
|
|||
it('should generate a functional snippet', async () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would be cool to have some sort of mocking framework/library for easily testing these plugins
Co-authored-by: Kanad Gupta <8854718+kanadgupta@users.noreply.github.com>
🧰 Changes
The dynamic, Proxy-based, variant of
api
has been really holding us back so this is the formal deprecation and removal of all code relating to it. It was a neat experimental hack that I am very proud of but it's time to move on to only supporting codegen here as all that work, and the TS type backings it offers, are far superior.Rest in peace little buddy.
fixes RM-7986