Skip to content

Drop nodejs build dependency

Tan Le requested to merge remove-node-js-deps into main

What does this merge request do and why?

We do not need nodejs to manually build tree-sitter typescript grammars since it is already handled by tree-sitter Python lib.

Similar MR on AI Gateway (gitlab-org/modelops/applied-ml/code-suggestions/ai-assist!460 (merged))

After this MR is merged, we can run rtx prune nodejs to remove the version previous installed by Prompt Library and save some precious disk spaces.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

Merge request checklist

  • I've ran the affected pipeline(s) to validate that nothing is broken.
  • Tests added for new functionality. If not, please raise an issue to follow up.
  • Documentation added/updated, if needed.
Edited by Tan Le

Merge request reports