Skip to content

Conversation

karlaalkhouri
Copy link

@karlaalkhouri karlaalkhouri commented Mar 5, 2025

Description

Mix multiple booster projects into a single one.

Changes

  • Created a script named repo-mixing.js in the directory scripts that merges all the .js files from all the projects altogether and puts them in /dist/repo-mixing.js inside the root project directory
  • Added a package.json file inside the root project directory

Checks

  • Run npm install
  • Run npm run mix to merge the projects

Additional information

This PR is for the issue #1514

Copy link

what-the-diff bot commented Mar 5, 2025

PR Summary

  • Introduction of Project Metadata File
    A new file named package.json has been created. This is like the ID card of our project, it contains information like the project's name, version, and a brief description. It also has instructions for building the project using our TypeScript and ESLint tools, so this file basically tells our tools what to do.

  • Development of a Script for Merging Files
    A new function named repo-mixing.js has been added. Its function is to combine multiple JavaScript files or documents from different folders that already have the package.json file into one single file. It's like gathering parts of a puzzle scattered in different boxes and compiling them together. This function also has its housekeeping duties, it can:

    • Create a new folder if one doesn't exist where it should
    • Clean up old data before merging new parts
    • Look for all the different pieces in directories with a package.json and put all .js files together, all the while keeping track of what it is doing.

@davidverdu
Copy link
Collaborator

Thank you for your PR @karlaalkhouri . We have the concern that Booster relies on Rush to manage packages and you are suggesting that it coexist with npm. The issue is that they could interfere with each other. Could you integrate your solution with Rush, maybe as a Rush command?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants