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.

Sign up for GitHub

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

Jump to bottom

build: migrate codegen to TS transformers #6141

Draft

AlCalzone wants to merge 1 commit into master from live-codegen

Member

AlCalzone commented Aug 10, 2023

When done, this will allow us to go back to simply invoking TypeScript to build the entire project.

Further reading: nonara/ts-patch#29


          build: start working on transformer-based codegen

cf64f63

github-advanced-security bot found potential problems

View reviewed changes

packages/ts-plugins/src/index.ts

+              		position: number,
+              	): tslib.Node | undefined {
+              		function find(node: tslib.Node): tslib.Node | undefined {
+              			const text = sourceFile.text.slice(node.getStart(), node.getEnd());

Check notice

Code scanning / CodeQL

Unused variable, import, function or class Note

Unused variable text.

packages/ts-plugins/src/index.ts

+              	function create(info: tslib.server.PluginCreateInfo) {
+              		// Get a list of things to remove from the completion list from the config object.
+              		// If nothing was specified, we'll just remove 'caller'
+              		const whatToRemove: string[] = info.config.remove || ["caller"];

Check notice

Code scanning / CodeQL

Unused variable, import, function or class Note

Unused variable whatToRemove.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet